Skip to content

fix(shared): dismiss notifications page push banner#5790

Merged
idoshamun merged 1 commit intomainfrom
eng-1112-feedback-bug-report-close-button-on-notifications-does-not
Mar 24, 2026
Merged

fix(shared): dismiss notifications page push banner#5790
idoshamun merged 1 commit intomainfrom
eng-1112-feedback-bug-report-close-button-on-notifications-does-not

Conversation

@idoshamun
Copy link
Copy Markdown
Member

@idoshamun idoshamun commented Mar 24, 2026

Summary

  • make the notifications page push CTA close button dismiss the banner reliably
  • keep overflow clipping for other notification CTA sources while allowing the notifications page close button to remain fully interactive
  • add a focused component test covering both CTA variants and the close button semantics

Key Decisions

  • set CloseButton to type="button" instead of changing the shared Button default to avoid a wider blast radius
  • scope the overflow change to the notifications page source only, so other banner layouts keep their existing clipping behavior

Closes ENG-1112


Created by Huginn 🐦‍⬛

Preview domain

https://eng-1112-feedback-bug-report-clo.preview.app.daily.dev

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Mar 24, 2026 1:09pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Mar 24, 2026 1:09pm

Request Review

@idoshamun idoshamun merged commit d35d603 into main Mar 24, 2026
12 checks passed
@idoshamun idoshamun deleted the eng-1112-feedback-bug-report-close-button-on-notifications-does-not branch March 24, 2026 15:01
idoshamun added a commit that referenced this pull request Mar 24, 2026
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