-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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(web): open image in new tab with memories on firefox #5847
Conversation
Deploying with Cloudflare Pages
|
Whhhhhhhhy do we need more z index changes.... |
You want to be able to click on the button when navigating your memories. I think we also use z indexes for navigation buttons on the asset-viewer. |
Then why can't we draw them on top? Z-index is for overwriting the default stacking order. Why does the default order not work? We should try to fix that if possible instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't test it myself, but the code seems better.
Like you suggested, buttons come after the image to avoid using z-index |
* fix: open image in new tab with memories on firefox * don't use z-index --------- Co-authored-by: Alex <alex.tran1502@gmail.com>
fixes #5751