Skip to content

fix (Alerts): Alert dialog dimension selection and dark mode visibility#8764

Merged
ericpgreen2 merged 2 commits intomainfrom
fix/alerts-ux-issues
Feb 4, 2026
Merged

fix (Alerts): Alert dialog dimension selection and dark mode visibility#8764
ericpgreen2 merged 2 commits intomainfrom
fix/alerts-ux-issues

Conversation

@ericpgreen2
Copy link
Contributor

@ericpgreen2 ericpgreen2 commented Feb 4, 2026

Addresses items 1 and 3 of this Slack report

Closes APP-709

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@ericpgreen2 ericpgreen2 self-assigned this Feb 4, 2026
@ericpgreen2 ericpgreen2 changed the title fix (Alerts): Handle edge case in initializing alerts form fix (Alerts): Alert dialog dimension selection and dark mode visibility Feb 4, 2026
@ericpgreen2 ericpgreen2 requested review from djbarnwal and removed request for AdityaHegde February 4, 2026 13:03
@ericpgreen2 ericpgreen2 merged commit 540d375 into main Feb 4, 2026
16 checks passed
@ericpgreen2 ericpgreen2 deleted the fix/alerts-ux-issues branch February 4, 2026 19:50
ericpgreen2 added a commit that referenced this pull request Feb 4, 2026
…ty (#8764)

* Fix edge case for initialization of alert's "split by" dimension

* Fix tabs numbers (hidden by accident from dark mode work)
ericpgreen2 added a commit that referenced this pull request Feb 4, 2026
…ty (#8764)

* Fix edge case for initialization of alert's "split by" dimension

* Fix tabs numbers (hidden by accident from dark mode work)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants