Skip to content

Keep preview from bleeding through the tags dropdown - #5193

Merged
connortechnology merged 1 commit into
ZoneMinder:masterfrom
Simpler1:patch-8
Oct 7, 2026
Merged

connortechnology merged 1 commit into
ZoneMinder:masterfrom
Simpler1:patch-8

Conversation

@Simpler1

@Simpler1 Simpler1 commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Removed z-index property from #frames img.

I'm not sure if this will have any other adverse effects, but I haven't found any.

When there are many tags in the dropdown list, the previews bleed through the list and make the tags unselectable where the previews are. This change allows the tags dropdown to display over the previews as they should.

Removed z-index property from #frames img.

I'm not sure if this will have any other adverse effects, but I haven't found any.  

When there are many tags in the dropdown list, the previews bleed through the list and make the tags unselectable where the previews are.  This change allows the tags dropdown to display over the previews as they should.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The focused CSS change resolves the stacking conflict without affecting thumbnail layout.

0 open findings

What changed in this PR

Removes the thumbnail stacking priority that caused event previews to overlap the tags dropdown.

Changes:

  • Removes z-index: 1000 from event preview images.
File Description
web/​skins/​classic/​css/​base/​views/​event.css Allows the tags dropdown to render above preview thumbnails.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@connortechnology
connortechnology merged commit c933812 into ZoneMinder:master Oct 7, 2026
10 checks passed
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.

3 participants