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

Header scrolls off screen #865

Closed
tarnung opened this issue Jun 28, 2022 · 3 comments · Fixed by #870
Closed

Header scrolls off screen #865

tarnung opened this issue Jun 28, 2022 · 3 comments · Fixed by #870
Labels
bug Something isn't working

Comments

@tarnung
Copy link
Collaborator

tarnung commented Jun 28, 2022

The header (where the undo/redo buttons sit) scrolls off screen when scrolling in a view.

This behaviour existed way back when but for the last two years the header was sticky if i recall correctly.

Feel free to close this if it's an intended change that i just happened not to like.

@tarnung tarnung added the bug Something isn't working label Jun 28, 2022
@munen
Copy link
Collaborator

munen commented Jun 30, 2022

Good catch.

It is not an intended change. I botched it by accident by implementing feat: Proper Landing Page #809 which was way more effort than I anticipated. But it also did bring a lot of goodies - not just the LP, but some refactoring. One PR before that, the header was still sticky.

My apologies 🙇 I remember that the sticky header was one of your contributions. It's certainly a UX regression and was not intended.

@munen
Copy link
Collaborator

munen commented Jun 30, 2022

Hm.. it's not as easy as I hoped. Before introducing #809, it worked. After introducing #809, it did not.

But going through the CSS code, I don't see what botched it. So it's probably markup related, not just CSS.

@munen
Copy link
Collaborator

munen commented Jun 30, 2022

I have created a PR based on the first original sticky header from 2019. That one turned out not to be sufficient since there were follow up PRs. However, it does seem to work well for me in first tests.

I'd be happy if you can check it out. If you like it, feel free to merge the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants