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

notebook container div still differs from "classical" Notebook experience #6388

Closed
edthrn opened this issue Jan 23, 2022 · 10 comments
Closed

Comments

@edthrn
Copy link

edthrn commented Jan 23, 2022

1. Problem

Even though both UIs are currently very close, I still feel frustration when using RetroLab, mostly because of CSS discrepancies.

With the classical Notebook interface, the notebook <div id="notebook-container"> kept left and right margin on small width windows, and increased in height as cells count grew.

It appears it is not the case with RetroLab latest version.

2. Proposed Solution

I would like to see the exact same UI on this notebook-container div, which is probably the most important of the whole document ; especially knowing that RetroLab will be used as the next major version of the Notebook.

Unfortunately, my front-end engineering skills are very poor, so I won't be able to propose a PR myself 😕

3. Screen shots to illustrate the difference

Both screenshots are made with the same (small) window width.

3.1 "Good ol' Notebook" has margins that makes the experience neater

image


3.2 Retro Lab has no margins on small width screens

image

@welcome
Copy link

welcome bot commented Jan 23, 2022

Thank you for opening your first issue in this project! Engagement like this is essential for open source projects! 🤗

If you haven't done so already, check out Jupyter's Code of Conduct. Also, please try to follow the issue template as it helps other other community members to contribute more effectively.
welcome
You can meet the other Jovyans by joining our Discourse forum. There is also an intro thread there where you can stop by and say Hi! 👋

Welcome to the Jupyter community! 🎉

@edthrn edthrn changed the title notebook container div of UI still differs from "classical" Notebook experience notebook container div still differs from "classical" Notebook experience Jan 23, 2022
@jtpio
Copy link
Member

jtpio commented Jan 24, 2022

Thanks @edthrn for opening this issue 👍

Indeed the styling of the Notebook component in RetroLab can still be improved.

Currently the "documented oriented feel" is implemented in CSS here:

https://github.com/jupyterlab/retrolab/blob/7fed46857663bb3f9e7ead599aaa85eefddf6c4e/packages/notebook-extension/style/base.css#L9-L46

Probably this could be tweaked to still keep some margins on smaller viewports.

@edthrn
Copy link
Author

edthrn commented Jan 25, 2022

Thanks for your reply 👍

Indeed, side margins on smaller viewports should be tweaked, but even more important is adjustable bottom margin for any width: it's a visually pleasant feature of the old notebook style.

@jtpio
Copy link
Member

jtpio commented Apr 27, 2022

Some improvements towards this were done in #6376, and are available in notebook>=7.0.0a3.

@jtpio jtpio transferred this issue from jupyterlab/retrolab Apr 27, 2022
@jtpio jtpio added this to the 7.0 milestone Apr 27, 2022
@ShobhanSrivastava
Copy link

ShobhanSrivastava commented Dec 21, 2022

Hey @jtpio is the issue still open? I would like to work on it

@jtpio
Copy link
Member

jtpio commented Dec 22, 2022

Thanks @ShobhanSrivastava.

I think there might still be some work to be done on the left and right margins. So if you want to have a look that would be great, thanks!

@Gardezi1
Copy link

is this still open?

@jtpio
Copy link
Member

jtpio commented Feb 28, 2023

This is what it currently looks like with the latest state of main as of: 2e4886f

notebook-div.mp4

If folks are happy with this we could indeed close the issue. We can always open new ones to iterate more if needed.

@harshil202
Copy link

Hello, @jtpio does it require changes?
If is there any changes then I would like to do them.

@andrii-i
Copy link
Contributor

This is what it currently looks like with the latest state of main as of: 2e4886f

notebook-div.mp4
If folks are happy with this we could indeed close the issue. We can always open new ones to iterate more if needed.

#6388 (comment)

Closing as solved based on this comment and "thumbs up" reactions to it

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants