XKit-Rewritten icon indicating copy to clipboard operation
XKit-Rewritten copied to clipboard

Shorten Posts: Button intersects search box/dropdown (minor)

Open marcustyphoon opened this issue 1 year ago • 1 comments

Platform

MacOS 13.2.1 arm64

Browser

Chrome 121.0.6167.139, Safari 17.2.1

Addon version

0.23.3

Details

Screenshot 2024-02-19 at 7 46 50 PM Screenshot 2024-02-19 at 7 47 09 PM

Because the z-index property of both the shorten posts expand button/tags and the search box that floats over search result pages is 2, they intersect unexpectedly.

Should be fixable with isolation: isolate; on the article.

marcustyphoon avatar Feb 20 '24 03:02 marcustyphoon

Should be fixable with isolation: isolate; on the article.

This has the slight potential to do bad things. XKit's popup elements—the ones from Quick Reblog and Quick Tags—will appear broken if they're inside an element with this applied. It's not usually possible to click control buttons on shortened posts, though, since the expand button covers the post controls, so you generally have to go out of your way to experience this in practice (e.g. by vertically resizing the browser window), and clicking expand fixes it.

Tumblr's code, as far as I know, uses popper.js and glass container and whatnot rather than actually z-indexing a real child element of a post to make a popup, so this shouldn't cause any issues, but it's not impossible that that could happen in the future.

Or, you know, redpop could use a z-index that differs by more than 1. That's also an option.

marcustyphoon avatar Mar 29 '24 09:03 marcustyphoon