Skip to content

🐛 Added z-index to confetti used by hooks#1209

Merged
mariush2 merged 4 commits into
mainfrom
fix/confetti-z-index
Jan 26, 2026
Merged

🐛 Added z-index to confetti used by hooks#1209
mariush2 merged 4 commits into
mainfrom
fix/confetti-z-index

Conversation

@Rotshrek
Copy link
Copy Markdown
Contributor

Azure DevOps links

Description

This very small PR adds the z-index property at max level so the confetti shows up on top of everything.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a z-index property to the confetti component's styling to ensure the confetti animation displays on top of other page elements. The change addresses a bug where confetti was being obscured by other content.

Changes:

  • Added zIndex: 999999999 to the IMPERATIVE_STYLING constant in the Confetti component

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/molecules/Confetti/Confetti.constants.ts
Comment thread src/molecules/Confetti/Confetti.constants.ts
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 20, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 100% (🎯 100%) 17200 / 17200
🔵 Statements 100% (🎯 100%) 17200 / 17200
🔵 Functions 100% (🎯 100%) 1242 / 1242
🔵 Branches 100% (🎯 100%) 4318 / 4318
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/molecules/Confetti/Confetti.constants.ts 100% 100% 100% 100%
Generated in workflow #2087 for commit dd84d59 by the Vitest Coverage Report Action

@mariush2
Copy link
Copy Markdown
Contributor

Do you need this to be released now? If so you can bump the patch version in this PR 😄

Copilot AI review requested due to automatic review settings January 26, 2026 07:29
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Rotshrek
Copy link
Copy Markdown
Contributor Author

Do you need this to be released now? If so you can bump the patch version in this PR 😄

Will do

@mariush2 mariush2 merged commit 19391f0 into main Jan 26, 2026
11 checks passed
@mariush2 mariush2 deleted the fix/confetti-z-index branch January 26, 2026 08:49
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