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

Fix left-nav height and padding #1661

Merged
merged 3 commits into from
Aug 18, 2023

Conversation

chalin
Copy link
Collaborator

@chalin chalin commented Aug 11, 2023

Possible followup change to be addressed:


Screenshots - sidebar_search_disable:true

Notes:

  • lg and md screenshots were taken with Firefox
  • "After" images are from my local build, where I set sidebar_search_disable:true

View: lg - notice the full sidenav height in the "After" image, and the top padding fix

The "Before" image is from the current production server, but with the search-box manually hidden.

Before After
image image

View: md - notice the full sidenav height in the "After" image

Before After
image image

View: sm - no change here (other than a tweak to the bottom padding

Before After
image image

Screenshots - sidebar_search_disable:false

View: lg - notice the full sidenav height in the "After" image

Before After
image image

View: md - notice the full sidenav height in the "After" image

Before After
image image

View: sm - no change here (other than a tweak to the bottom padding

Before After
image image

@chalin chalin changed the title Fix left-nav height and padding (no vars) Fix left-nav height and padding Aug 11, 2023
@chalin chalin force-pushed the chalin-im-left-nav-fix-2023-08-11 branch from 413eb0f to cff5757 Compare August 17, 2023 10:16
@chalin
Copy link
Collaborator Author

chalin commented Aug 17, 2023

@yann-soubeyrand - might you be available to review and or comment on this PR? Thanks.

@yann-soubeyrand
Copy link
Contributor

@yann-soubeyrand - might you be available to review and or comment on this PR? Thanks.

Sure, but I’m clearly not a CSS guru: I just fixed two small issues, which took me a lot of time due to my CSS skill! 😆

Anyway, this looks good to me and the visuals are convincing.

@thisisobate
Copy link

Works as expected!

@chalin chalin force-pushed the chalin-im-left-nav-fix-2023-08-11 branch from cff5757 to 6652fd3 Compare August 17, 2023 22:05
Copy link
Collaborator

@deining deining left a comment

Choose a reason for hiding this comment

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

LGTM.

@chalin
Copy link
Collaborator Author

chalin commented Aug 18, 2023

Thanks all for your feedback.

@chalin chalin merged commit 5295589 into google:main Aug 18, 2023
5 checks passed
@chalin chalin deleted the chalin-im-left-nav-fix-2023-08-11 branch August 18, 2023 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants