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

Cells aren't highlighted when revealing in outline #122890

Closed
roblourens opened this issue May 4, 2021 · 6 comments
Closed

Cells aren't highlighted when revealing in outline #122890

roblourens opened this issue May 4, 2021 · 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 notebook-markdown verified Verification succeeded
Milestone

Comments

@roblourens
Copy link
Member

  • Open a notebook
  • Open the outline and use arrow keys
  • Cells are revealed when selected in the outline but previously they got a yellow bg highlight, now there is no highlight

Maybe related to #122720 somehow

@jrieken
Copy link
Member

jrieken commented May 4, 2021

Cells are revealed when selected in the outline but previously they got a yellow bg highlight, now there is no highlight

Only quick outline and breadcrumbs do that. The outline view doesn't do this - afaik never did. @roblourens what which outline do you mean? view or quick?

@roblourens
Copy link
Member Author

roblourens commented May 4, 2021

Quick outline and breadcrumbs. Today I see that I get the background highlight on code cells but not markdown cells. Was only using markdown notebook last night. I don't know whether we highlighted markdown cells previously.

@roblourens roblourens assigned rebornix and unassigned jrieken May 4, 2021
@jrieken
Copy link
Member

jrieken commented May 5, 2021

Quick outline and breadcrumbs. Today I see that I get the background highlight on code cells but not markdown cells.

Yeah, I am also seeing that (and have files a dupe issue ;-))

@rebornix
Copy link
Member

rebornix commented May 5, 2021

I think it's an issue with the custom markdown renderer. I fixed this before but not sure why it regressed now.

@rebornix
Copy link
Member

rebornix commented May 5, 2021

it's the Shadow DOM :) We applied class names on the container, but now the markdown previews are rendered in the Shadow DOM and styles will not be applied

case 'decorations':
{
const outputContainer = document.getElementById(event.data.cellId);
outputContainer?.classList.add(...event.data.addedClassNames);
outputContainer?.classList.remove(...event.data.removedClassNames);
}
break;

@rebornix rebornix removed their assignment May 5, 2021
@rebornix rebornix added the bug Issue identified by VS Code Team member as probable bug label May 5, 2021
@mjbvz mjbvz closed this as completed in 65a0ac6 May 5, 2021
@connor4312 connor4312 added the verified Verification succeeded label Jun 3, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Jun 19, 2021
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 notebook-markdown verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

6 participants
@roblourens @rebornix @jrieken @connor4312 @mjbvz and others