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 ribbon going over sticky tiddler title #8096

Open
wants to merge 1 commit into
base: tiddlywiki-com
Choose a base branch
from

Conversation

Telumire
Copy link
Contributor

@Telumire Telumire commented Mar 21, 2024

isolation:isolate; is added to the style for the card in order to prevent the ribbon from going in front of the title bar while in position sticky.

before (without isolation:isolate;):
image

after (with isolation:isolate;):
image

I made a simplified demo of how isolation:isolate fix z-index issues: https://minimal-css.tiddlyhost.com/#snippet%2Ffixing%20z-index%2Fisolation

Copy link

vercel bot commented Mar 21, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
tiddlywiki5 ✅ Ready (Inspect) Visit Preview Mar 21, 2024 2:19am

@pmario
Copy link
Member

pmario commented Mar 21, 2024

Please check if there are existing PRs #8075

I did a complete rewrite, because the old one is completely bugged
IMO Hardcoded styles make everything worse

@Telumire
Copy link
Contributor Author

Telumire commented Mar 21, 2024

@pmario I did check the other PR, but the issue comes from the card container, not the ribbon itself. I'll check your rewrite to see if it solve the issue, and I'll close this PR if it's the case (edit: it doesn't)

IMO Hardcoded styles make everything worse

I agree ! I was wondering if there was a reason why this was done this way

@pmario
Copy link
Member

pmario commented Mar 21, 2024

Sorry my fault. I did see the fork-github-ribbon, but you posted the image of GettingStarted.

My linked PR has nothing to do with your PR . Sorry

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants