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

Layout of top ribbon for small desktop heights can overflow #595

Open
wbazant opened this issue Nov 14, 2024 · 1 comment · May be fixed by #598
Open

Layout of top ribbon for small desktop heights can overflow #595

wbazant opened this issue Nov 14, 2024 · 1 comment · May be fixed by #598
Labels
bug Unexpected problems or unintended behavior desktop Specific to desktop layout

Comments

@wbazant
Copy link
Collaborator

wbazant commented Nov 14, 2024

Minor issue, but there are widths where the current layout doesn't quite work. This is what a screen with 770px width looks like - note the Twitter icon going off, and a triangle in the 'about' button:
Screenshot from 2024-11-14 18-53-17

Screenshot from 2024-11-14 18-55-05

Maybe some clever layout changes will fix it - for example there's lots of space around the icon? If impossible, we can increase minimal width for desktop screens, e.g. from 767 to 800 px, but we'd like to understand how many pixels we need and for what.

@wbazant wbazant added bug Unexpected problems or unintended behavior desktop Specific to desktop layout labels Nov 14, 2024
@ahmedhalac
Copy link
Contributor

ahmedhalac commented Nov 15, 2024

Hi @wbazant. For me, this is only reproducible for French language.

For English it's as expected:

Screenshot 2024-11-15 at 10 03 40 PM

ahmedhalac added a commit to ahmedhalac/falling-fruit-web that referenced this issue Nov 15, 2024
ahmedhalac added a commit to ahmedhalac/falling-fruit-web that referenced this issue Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected problems or unintended behavior desktop Specific to desktop layout
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants