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

Notebooks: have a lot of unneeded margins/paddings? #101366

Closed
bpasero opened this issue Jun 30, 2020 · 13 comments
Closed

Notebooks: have a lot of unneeded margins/paddings? #101366

bpasero opened this issue Jun 30, 2020 · 13 comments
Assignees
Labels
notebook verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@bpasero
Copy link
Member

bpasero commented Jun 30, 2020

Testing #100965

image

I find this could be more condense.

@miguelsolorio
Copy link
Contributor

This was intentional, previously code cells and markdown cells were on the same vertical plane and it was hard to tell which cells were code vs markdown (especially when edited a markdown cell). So we decided to indent code cells to give better hierarchy.

We could tighten the left margin by a few pixels (~8px) but we'd still need to keep the code cells indented.

@roblourens
Copy link
Member

@bpasero are you thinking about horizontal padding or vertical padding or both?

@bpasero
Copy link
Member Author

bpasero commented Jul 1, 2020

Both horizontal and vertical padding! Maybe have a setting where I can decide if I want the condense view or not.

Now that I look at my screenshot from #101366 (comment) again, it really feels like there is a bug in notebook that prevents some data to be shown.

@miguelsolorio
Copy link
Contributor

I've updated the overall padding/margin around the cells, between the cells, and inside the editors:

Frame 6

I also created #101635 to propose to hide the status bar until needed, and #101633 to explore a more compact bottom toolbar which will reduce more space.

Let me know if this looks good for you and we can close out this issue.

@bpasero
Copy link
Member Author

bpasero commented Jul 3, 2020

It is better, thanks for reducing it. I still think there is lots of whitespace, e.g. here:

image

The line around Config: defines the inbox query being in small font having almost 5x the line height as surrounding whitespace.

@bpasero
Copy link
Member Author

bpasero commented Jul 3, 2020

@rebornix @misolori is there any way I can see how this used to look in 1.46? How can I run an older notebook UX currently, if possible?

@roblourens
Copy link
Member

You can see it in stable, starting with --enable-proposed-api for whichever extension

@bpasero
Copy link
Member Author

bpasero commented Jul 4, 2020

@roblourens this results in:

Unable to open 'inbox.github-issues': Cannot read property 'backupId' of undefined.

@roblourens
Copy link
Member

Oh, I guess you would have to use an older build of the extension. It works for the markdown notebook.

@bpasero
Copy link
Member Author

bpasero commented Jul 4, 2020

Thanks that works 👍

@rebornix rebornix added this to the July 2020 milestone Jul 6, 2020
@rebornix
Copy link
Member

rebornix commented Jul 6, 2020

The editor inner left padding issue is extracted to #101808 .

@miguelsolorio
Copy link
Contributor

I think we've addressed a lot of these issues, below is a comparison from June => July:

image

Let us know if you think it's better.

@miguelsolorio miguelsolorio added the verification-needed Verification of issue is requested label Jul 23, 2020
@isidorn isidorn added the verified Verification succeeded label Aug 5, 2020
@isidorn
Copy link
Contributor

isidorn commented Aug 5, 2020

I see there are improvments, adding verified and Ben can open more issues once he is back if this did not capture everything he pointed out.

@github-actions github-actions bot locked and limited conversation to collaborators Sep 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
notebook verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants