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

Side by side notebooks and error highlights #106215

Closed
jrieken opened this issue Sep 7, 2020 · 6 comments
Closed

Side by side notebooks and error highlights #106215

jrieken opened this issue Sep 7, 2020 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders notebook verified Verification succeeded

Comments

@jrieken
Copy link
Member

jrieken commented Sep 7, 2020

It seems like reliable steps are:

  1. Have notebook1 with problems open in the left hand side and nodebook2 open on the right
  2. Put your focus in notebook2
  3. Click on the problems view
  4. notebook1 will open on the right hand side, but the line/code cell will not be highlighted 🐛

Originally posted by @connor4312 in #104454 (comment)

@jrieken jrieken self-assigned this Sep 7, 2020
@jrieken jrieken added this to the September 2020 milestone Sep 7, 2020
@jrieken jrieken added info-needed Issue requires more information from poster notebook labels Sep 14, 2020
@jrieken
Copy link
Member Author

jrieken commented Sep 14, 2020

Those steps don't reproduce this for me. @connor4312 please check again or refine steps

Sep-14-2020 15-34-43

@connor4312
Copy link
Member

Here's a video of what I'm seeing: https://memes.peet.io/img/20-09-c7c7e81b-d4a4-4696-b1c7-cf707dd7e283.mp4

It seems like the first open actually does focus the cell, but not the line. This is particularly noticable if you have a long code cell where the problem might not even be in view on the first click.

@connor4312 connor4312 reopened this Sep 26, 2020
@jrieken jrieken removed the info-needed Issue requires more information from poster label Oct 1, 2020
@jrieken jrieken added the bug Issue identified by VS Code Team member as probable bug label Oct 1, 2020
@jrieken jrieken closed this as completed in 35d4bbf Oct 1, 2020
@jrieken jrieken reopened this Oct 1, 2020
@jrieken
Copy link
Member Author

jrieken commented Oct 1, 2020

This works better not not really good.

@jrieken
Copy link
Member Author

jrieken commented Oct 1, 2020

@rebornix I am now using revealLineInCenterIfOutsideViewportAsync which seems very promising and it does fix the original bug but when having a simpler case it doesn't work. Do this

  1. have a large code cell (see below)
  2. make sure the error isn't visible
  3. select the error in the errors pane
  4. 🐛 the line isn't scrolled into view
$rpo=repo:microsoft/vscode


































$repo is:open milestone:"April 2020" assignee:@me label:*english-please

@rebornix
Copy link
Member

rebornix commented Oct 1, 2020

Verifier: please test #106215 (comment) .

@bpasero bpasero added the verified Verification succeeded label Oct 2, 2020
msftwindowslinux pushed a commit to msftwindowslinux/vscode that referenced this issue Oct 2, 2020
msftwindowslinux pushed a commit to msftwindowslinux/vscode that referenced this issue Oct 2, 2020
msftwindowslinux pushed a commit to msftwindowslinux/vscode that referenced this issue Oct 2, 2020
msftwindowslinux pushed a commit to msftwindowslinux/vscode that referenced this issue Oct 2, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders notebook verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants
@rebornix @bpasero @jrieken @connor4312 and others