Skip to content
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

Editor selection is not unset while navigating from Problems view #99136

Closed
sandy081 opened this issue Jun 2, 2020 · 6 comments
Closed

Editor selection is not unset while navigating from Problems view #99136

sandy081 opened this issue Jun 2, 2020 · 6 comments
Assignees
Labels
notebook under-discussion Issue is under discussion for relevance, priority, approach
Milestone

Comments

@sandy081
Copy link
Member

sandy081 commented Jun 2, 2020

Testing #98821

  • Have following issue notebook
[{"kind":2,"language":"github-issues","value":"repo:microsoft/vscode assigne:@me label:settings","editable":true},{"kind":2,"language":"github-issues","value":"rep:microsoft/vscode assignee:@me label=1234","editable":true}]
  • Open the notebook
  • Navigate the problems in problems views using arrows
  • Selection is not unset when navigating across cells. For eg: See that in the screenshot selection is set in two cells

image

@jrieken jrieken added bug Issue identified by VS Code Team member as probable bug notebook labels Jun 3, 2020
@jrieken
Copy link
Member

jrieken commented Jun 3, 2020

This is likely an issue with selectElement

@jrieken jrieken added this to the May 2020 milestone Jun 3, 2020
@rebornix
Copy link
Member

rebornix commented Jun 3, 2020

Good catch and yet another case for composite editor. Image you have two editors side by side and each one of them has a problem

recording (1)

and navigating between the problems will not unset the selection of the previous editor. The problems panel will only set new selection in an editor, thus if you are navigating problems in a single editor, the selection is always updated.

However in our scenario, we have composite editors in one notebook and those composite editors don't share selections. And also since the problems panel doesn't unset selections of previous cells, the selections are left over. The same behavior applies to Find All References, Find & Replace, etc.

@rebornix rebornix modified the milestones: May 2020, June 2020 Jun 3, 2020
@rebornix rebornix added under-discussion Issue is under discussion for relevance, priority, approach and removed bug Issue identified by VS Code Team member as probable bug labels Jun 3, 2020
@jrieken
Copy link
Member

jrieken commented Jun 15, 2020

I have pushed a change that makes the selection color almost transparent when a cell isn't selected anymore. I think that's better than unsetting the selection of an inactive editor. Let me know what you think.

Jun-15-2020 16-00-35

@jrieken
Copy link
Member

jrieken commented Jun 15, 2020

⬆️ fyi @misolori

@miguelsolorio
Copy link
Contributor

I think this is fine for now, once we support cell background colors I'd imagine we'd move towards that to match the selection in the editor.

@jrieken jrieken closed this as completed Jun 16, 2020
@jrieken
Copy link
Member

jrieken commented Jun 16, 2020

Thanks. I'll leave it like that then

@github-actions github-actions bot locked and limited conversation to collaborators Aug 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
notebook under-discussion Issue is under discussion for relevance, priority, approach
Projects
None yet
Development

No branches or pull requests

5 participants