Skip to content

Commit b2eb9b1

Browse files
authored
Don't test for cursor position on mousemove if the widget adapter is not connected (#1114)
* Noisy log message Fixes #1113 * Add changelog entry * Lint and remove duplicate --------- Co-authored-by: Frédéric Collonval <[email protected]>
1 parent f5d2b9e commit b2eb9b1

File tree

4 files changed

+29
-30
lines changed

4 files changed

+29
-30
lines changed

.github/workflows/job.test.yml

+3-3
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ jobs:
6262
uses: conda-incubator/setup-miniconda@v3
6363
with:
6464
environment-file: requirements/github-actions.yml
65-
mamba-version: "*"
65+
mamba-version: '*'
6666
channels: conda-forge,nodefaults
6767
channel-priority: true
6868

@@ -138,7 +138,7 @@ jobs:
138138
- name: Set up Python and mamba
139139
uses: conda-incubator/setup-miniconda@v3
140140
with:
141-
mamba-version: "*"
141+
mamba-version: '*'
142142
channels: conda-forge,nodefaults
143143
channel-priority: true
144144

@@ -239,7 +239,7 @@ jobs:
239239
with:
240240
python-version: ${{ matrix.python }}
241241
environment-file: requirements/github-actions.yml
242-
mamba-version: "*"
242+
mamba-version: '*'
243243
channels: conda-forge,nodefaults
244244
channel-priority: true
245245

CHANGELOG.md

+5
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
## Changelog
22

3+
### `@jupyter-lsp/jupyterlab-lsp (next)`
4+
5+
- bug fixes:
6+
- Don't look at editor under the mouse on mouse motion when the widget adapter is not connected ([#1113](https://github.com/jupyter-lsp/jupyterlab-lsp/issues/1113))
7+
38
### `jupyter-lsp 2.2.5`
49

510
- bug fixes:

packages/jupyterlab-lsp/src/features/hover.ts

+4
Original file line numberDiff line numberDiff line change
@@ -194,6 +194,10 @@ export class HoverFeature extends Feature {
194194
});
195195
const eventListeners = EditorView.domEventHandlers({
196196
mousemove: event => {
197+
// Bail early if the adapter is not connected - and therefore no tooltip can be displayed
198+
if (!adapter.isConnected) {
199+
return;
200+
}
197201
// this is used to hide the tooltip on leaving cells in notebook
198202
this.updateUnderlineAndTooltip(event, adapter)
199203
?.then(keepTooltip => {

yarn.lock

+17-27
Original file line numberDiff line numberDiff line change
@@ -16799,20 +16799,13 @@ __metadata:
1679916799
languageName: node
1680016800
linkType: hard
1680116801

16802-
"vscode-jsonrpc@npm:8.0.2, vscode-jsonrpc@npm:^8.0.2":
16802+
"vscode-jsonrpc@npm:8.0.2":
1680316803
version: 8.0.2
1680416804
resolution: "vscode-jsonrpc@npm:8.0.2"
1680516805
checksum: 9d055fd4c87ef1093b0eecb5370bfaf3402179b6639149b6d0f7e0bde60cf580091c7e07b0caff868f10f90331b17e7383c087217c077fdd1b5ae7bc23b72f77
1680616806
languageName: node
1680716807
linkType: hard
1680816808

16809-
"vscode-jsonrpc@npm:8.2.0":
16810-
version: 8.2.0
16811-
resolution: "vscode-jsonrpc@npm:8.2.0"
16812-
checksum: f302a01e59272adc1ae6494581fa31c15499f9278df76366e3b97b2236c7c53ebfc71efbace9041cfd2caa7f91675b9e56f2407871a1b3c7f760a2e2ee61484a
16813-
languageName: node
16814-
linkType: hard
16815-
1681616809
"vscode-jsonrpc@npm:^4.0.0":
1681716810
version: 4.0.0
1681816811
resolution: "vscode-jsonrpc@npm:4.0.0"
@@ -16827,6 +16820,13 @@ __metadata:
1682716820
languageName: node
1682816821
linkType: hard
1682916822

16823+
"vscode-jsonrpc@npm:^8.0.2":
16824+
version: 8.2.0
16825+
resolution: "vscode-jsonrpc@npm:8.2.0"
16826+
checksum: f302a01e59272adc1ae6494581fa31c15499f9278df76366e3b97b2236c7c53ebfc71efbace9041cfd2caa7f91675b9e56f2407871a1b3c7f760a2e2ee61484a
16827+
languageName: node
16828+
linkType: hard
16829+
1683016830
"vscode-languageclient@npm:^6.1.3":
1683116831
version: 6.1.4
1683216832
resolution: "vscode-languageclient@npm:6.1.4"
@@ -16887,7 +16887,7 @@ __metadata:
1688716887
languageName: node
1688816888
linkType: hard
1688916889

16890-
"vscode-languageserver-protocol@npm:3.17.2, vscode-languageserver-protocol@npm:^3.10.3, vscode-languageserver-protocol@npm:^3.15.3, vscode-languageserver-protocol@npm:^3.17.0, vscode-languageserver-protocol@npm:^3.7.2":
16890+
"vscode-languageserver-protocol@npm:3.17.2, vscode-languageserver-protocol@npm:^3.10.3, vscode-languageserver-protocol@npm:^3.15.0, vscode-languageserver-protocol@npm:^3.15.3, vscode-languageserver-protocol@npm:^3.17.0, vscode-languageserver-protocol@npm:^3.7.2":
1689116891
version: 3.17.2
1689216892
resolution: "vscode-languageserver-protocol@npm:3.17.2"
1689316893
dependencies:
@@ -16907,16 +16907,6 @@ __metadata:
1690716907
languageName: node
1690816908
linkType: hard
1690916909

16910-
"vscode-languageserver-protocol@npm:^3.15.0":
16911-
version: 3.17.5
16912-
resolution: "vscode-languageserver-protocol@npm:3.17.5"
16913-
dependencies:
16914-
vscode-jsonrpc: 8.2.0
16915-
vscode-languageserver-types: 3.17.5
16916-
checksum: dfb42d276df5dfea728267885b99872ecff62f6c20448b8539fae71bb196b420f5351c5aca7c1047bf8fb1f89fa94a961dce2bc5bf7e726198f4be0bb86a1e71
16917-
languageName: node
16918-
linkType: hard
16919-
1692016910
"vscode-languageserver-textdocument@npm:1.0.8, vscode-languageserver-textdocument@npm:^1.0.1":
1692116911
version: 1.0.8
1692216912
resolution: "vscode-languageserver-textdocument@npm:1.0.8"
@@ -16966,27 +16956,27 @@ __metadata:
1696616956
languageName: node
1696716957
linkType: hard
1696816958

16969-
"vscode-languageserver-types@npm:3.17.2, vscode-languageserver-types@npm:^3.13.0, vscode-languageserver-types@npm:^3.16.0, vscode-languageserver-types@npm:^3.17.0-next.3, vscode-languageserver-types@npm:^3.7.2":
16959+
"vscode-languageserver-types@npm:3.17.2":
1697016960
version: 3.17.2
1697116961
resolution: "vscode-languageserver-types@npm:3.17.2"
1697216962
checksum: ef2d862d22f622b64de0f428773d50a5928ec6cdd485960a7564ebe4fd4a3c8bcd956f29eb15bc45a0f353846e62f39f6c764d2ab85ce774b8724411ba84342f
1697316963
languageName: node
1697416964
linkType: hard
1697516965

16976-
"vscode-languageserver-types@npm:3.17.5":
16977-
version: 3.17.5
16978-
resolution: "vscode-languageserver-types@npm:3.17.5"
16979-
checksum: 79b420e7576398d396579ca3a461c9ed70e78db4403cd28bbdf4d3ed2b66a2b4114031172e51fad49f0baa60a2180132d7cb2ea35aa3157d7af3c325528210ac
16980-
languageName: node
16981-
linkType: hard
16982-
1698316966
"vscode-languageserver-types@npm:3.5.0":
1698416967
version: 3.5.0
1698516968
resolution: "vscode-languageserver-types@npm:3.5.0"
1698616969
checksum: 44c859700cfda71b5d3ff1224b5a1df093b39ec6755c8beb12fc76d1fbdb1a4948f55c3a97768356a1cbfb4e3ce66c5e4380e25f66bafcbd2db50277f68ee989
1698716970
languageName: node
1698816971
linkType: hard
1698916972

16973+
"vscode-languageserver-types@npm:^3.13.0, vscode-languageserver-types@npm:^3.16.0, vscode-languageserver-types@npm:^3.17.0-next.3, vscode-languageserver-types@npm:^3.7.2":
16974+
version: 3.17.5
16975+
resolution: "vscode-languageserver-types@npm:3.17.5"
16976+
checksum: 79b420e7576398d396579ca3a461c9ed70e78db4403cd28bbdf4d3ed2b66a2b4114031172e51fad49f0baa60a2180132d7cb2ea35aa3157d7af3c325528210ac
16977+
languageName: node
16978+
linkType: hard
16979+
1699016980
"vscode-languageserver@npm:8.0.0-next.8":
1699116981
version: 8.0.0-next.8
1699216982
resolution: "vscode-languageserver@npm:8.0.0-next.8"

0 commit comments

Comments
 (0)