Menu

#534 fix(ui): stop StatusBanner clicks from bubbling to the dialog overlay

closed
nobody
None
2026-07-21
2026-07-21
Anonymous
No

Originally created by: Akarsh-Hegde

Summary

Fixes a CodeRabbit finding on Meridiona/meridian#528 (ui/components/timeline/TaskDetailDialog.tsx, Major/functional-correctness).

StatusBanner was rendered as a sibling of the modal body's onClick={e => e.stopPropagation()} div, inside the outer overlay's onClick={onClose}. Its Undo/dismiss buttons don't stop propagation, so any click on them bubbled straight to the overlay and closed the whole dialog — undermining the Undo flow that feature added.

Fix

Wrapped <StatusBanner /> in its own onClick={e => e.stopPropagation()} div, matching the reviewer's proposed fix exactly. StatusBanner itself renders as position: fixed, so it isn't affected by being nested one level deeper (no ancestor establishes a new containing block).

Test plan

  • [x] Full pre-push suite green: fmt, clippy, UI build, UI tests, cargo test

Related

Tickets: #528

Discussion

  • Anonymous

    Anonymous - 2026-07-21

    Originally posted by: coderabbitai[bot]

    [!IMPORTANT]

    Review skipped

    Auto reviews are disabled on base/target branches other than the default branch.

    Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.


    ⚙️ Run configuration

    Configuration used: Organization UI

    Review profile: ASSERTIVE

    Plan: Pro Plus

    Run ID: b48aaf62-3c2a-4ba5-bc5f-c05a85964ad0

    You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

    Use the checkbox below for a quick retry:
    - [ ] 🔍 Trigger review

    ✨ Finishing Touches
    🧪 Generate unit tests (beta) - [ ] Create PR with unit tests - [ ] Commit unit tests in branch `fix/status-banner-click-propagation`

    Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

    ❤️ Share - [X](https://twitter.com/intent/tweet?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A&url=https%3A//coderabbit.ai) - [Mastodon](https://mastodon.social/share?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A%20https%3A%2F%2Fcoderabbit.ai) - [Reddit](https://www.reddit.com/submit?title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&text=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code.%20Check%20it%20out%3A%20https%3A//coderabbit.ai) - [LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcoderabbit.ai&mini=true&title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&summary=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code)

    Comment @coderabbitai help to get the list of available commands.

     
  • Anonymous

    Anonymous - 2026-07-21

    Ticket changed by: Akarsh-Hegde

    • status: open --> closed
     

Log in to post a comment.