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

Enhancement: Added drop shadows to markdown, text, and tree views #6893

Merged

Conversation

rielAsh24
Copy link
Contributor

@rielAsh24 rielAsh24 commented May 30, 2023

As mentioned in #6781 , Markdown and Text views had edges without drop shadow, unlike the notebooks.

This fix works on adding the drop shadows to these views, and the Tree view/File browser.

@github-actions
Copy link
Contributor

Binder 👈 Launch a Binder on branch rielAsh24/notebook/fix_6781-Added_Drop-Shadows_Markdown_Text_Tree

@jtpio jtpio added this to the 7.0 milestone May 30, 2023
@rielAsh24 rielAsh24 changed the title Added drop shadows to markdown, text, and tree views Enhancement: Added drop shadows to markdown, text, and tree views May 30, 2023
@jtpio
Copy link
Member

jtpio commented May 30, 2023

Thanks @rielAsh24 for working on this!

@rielAsh24
Copy link
Contributor Author

Thanks @rielAsh24 for working on this!

Glad to contribute!

@jtpio
Copy link
Member

jtpio commented May 31, 2023

bot please update playwright snapshots

Adjusted drop shadows for tree view, markdown and text notebooks.
Tree View drop shadow values are in co-ordination with the others.
Adjusted main-panel padding to be responsive
Added data-format body selectors to the css
Re-wrote the media query for allowing responsiveness without a refresh
@rielAsh24 rielAsh24 force-pushed the fix_6781-Added_Drop-Shadows_Markdown_Text_Tree branch from 41f93e6 to ed28e9c Compare May 31, 2023 17:15
@jtpio
Copy link
Member

jtpio commented May 31, 2023

bot please update playwright snapshots

@jtpio jtpio closed this May 31, 2023
@jtpio jtpio reopened this May 31, 2023
Copy link
Member

@jtpio jtpio left a comment

Choose a reason for hiding this comment

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

Thanks!

@jtpio jtpio merged commit a605fa9 into jupyter:main Jun 1, 2023
24 checks passed
@rielAsh24
Copy link
Contributor Author

Thanks!

Thank you @jtpio for your encouragement and support. Was great working on this pull request! 😄

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.

Markdown and text documents do not have drop shadow
2 participants