-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fixed #12392. Tab indices now updated #12400
fixed #12392. Tab indices now updated #12400
Conversation
a98e2df
to
6c1e99e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still notice errors with this approach:
rpc-message-encoder.ts:184 Uncaught (in promise) Error: Tab close updated received for index 3 which does not exist
at TabGroupExt.acceptTabOperation (/workspace/theia/packages/plugin-ext/lib/plugin/tabs.js:156:23)
at TabsExtImpl.$acceptTabOperation (/workspace/theia/packages/plugin-ext/lib/plugin/tabs.js:286:27)
at /workspace/theia/packages/plugin-ext/lib/common/proxy-handler.js:91:71
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at async RpcProtocol.handleRequest (/workspace/theia/packages/core/lib/common/message-rpc/rpc-protocol.js:167:28)
Steps:
- open 4 editors
- close the third editor (2nd last)
- close the last editor
a02a1e7
to
9f1a51b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't seem to reproduce the error any longer 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can confirm that the error is no longer reported in the console 👍
9b20e39
to
964a1ab
Compare
b53e89d
to
b0fff47
Compare
b0fff47
to
2a39366
Compare
@vince-fugnitto merge this one? |
after deleting and moving a tab Signed-off-by: Jonah Iden <[email protected]>
Signed-off-by: Jonah Iden <[email protected]>
Co-authored-by: Mark Sujew <[email protected]>
ce564f0
to
023821c
Compare
after deleting and moving a tab
What it does
Fixes #12392
updates tabs indeces of all tabs in a group when a tab is deleted or moved
How to test
as described in the issue
Review checklist
Reminder for reviewers