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

Make the DOM structure of linked output views more similar to outputs #14911

Merged
merged 2 commits into from
Aug 9, 2023

Conversation

SylvainCorlay
Copy link
Member

This fixes #14564 by giving the linked output view the same DOM structure as the normal output (except for prompts).

@jupyterlab-probot
Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@SylvainCorlay
Copy link
Member Author

Ping @krassowski.

In this PR, I am just making the linked view of the output be more like the actual output in terms of DOM structure, which is probably better regardless of other questions.

I will open a PR on fixing the firefox issue after this one.

@krassowski krassowski added this to the 4.0.x milestone Aug 3, 2023
Copy link
Member

@krassowski krassowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified that this fixes the issue. The visual regression failures appear unrelated, I will merge the main branch to rerun with the recent fixes to CI.

@krassowski krassowski merged commit 4f6ed4c into jupyterlab:main Aug 9, 2023
77 checks passed
@krassowski
Copy link
Member

@meeseeksdev please backport to 4.0.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Aug 9, 2023
jtpio pushed a commit that referenced this pull request Aug 9, 2023
…e similar to outputs (#14941)

Co-authored-by: Sylvain Corlay <sylvain.corlay@gmail.com>
@SylvainCorlay SylvainCorlay deleted the linked-output-view branch August 11, 2023 09:27
@SylvainCorlay
Copy link
Member Author

Thanks, I will fix the scrolling issue now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Create new view for output" in jupyter lab 4.0.0 not working as before
2 participants