Skip to content

Let clicks pass through the object breadcrumbs popup (#9128)#10038

Open
dpage wants to merge 1 commit into
pgadmin-org:masterfrom
dpage:fix-9128-breadcrumbs-clicks
Open

Let clicks pass through the object breadcrumbs popup (#9128)#10038
dpage wants to merge 1 commit into
pgadmin-org:masterfrom
dpage:fix-9128-breadcrumbs-clicks

Conversation

@dpage

@dpage dpage commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes #9128.

The object breadcrumbs popup is an absolutely-positioned overlay (position: absolute; bottom: 0; zIndex: 1004) at the bottom-left of the object explorer. Because it's purely informational, it shouldn't capture mouse input — but it did, blocking clicks on the tree items underneath it. Added pointer-events: none to the styled container so clicks pass through.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation

    • Version 9.16 release notes now available with supported PostgreSQL and EDB Advanced Server versions, plus bundled PostgreSQL utilities information (psql/pg_dump/pg_restore at 18.2).
  • Bug Fixes

    • Fixed object breadcrumbs popup blocking user interactions in the object explorer.

@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@dpage, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 7 minutes and 28 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 20746027-aeee-4262-8cef-93836646ce1a

📥 Commits

Reviewing files that changed from the base of the PR and between 0752301 and 2c593e3.

📒 Files selected for processing (2)
  • docs/en_US/release_notes_9_16.rst
  • web/pgadmin/static/js/components/ObjectBreadcrumbs.jsx

Walkthrough

This PR fixes a UI bug where the ObjectBreadcrumbs popup blocks pointer interactions with tree items below it by adding pointerEvents: 'none' to the breadcrumbs styling. The fix is documented in a new Version 9.16 release notes page that lists supported database versions and references the bug resolution.

Changes

ObjectBreadcrumbs popup fix and documentation

Layer / File(s) Summary
ObjectBreadcrumbs pointer-events styling
web/pgadmin/static/js/components/ObjectBreadcrumbs.jsx
CSS styling adds pointerEvents: 'none' to the StyledBox component to prevent the breadcrumbs overlay from blocking interactions with underlying tree elements.
Version 9.16 release notes
docs/en_US/release_notes.rst, docs/en_US/release_notes_9_16.rst
New release notes page documents Version 9.16 with TBD release date, supported PostgreSQL and EDB Advanced Server versions, bundled utilities (psql/pg_dump/pg_restore at 18.2), and includes the ObjectBreadcrumbs bug fix entry referencing Issue #9128. The notes page is added to the documentation toctree.

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically summarizes the main code change: adding pointer-events handling to allow clicks to pass through the breadcrumbs popup, with the issue reference for context.
Linked Issues check ✅ Passed The PR directly addresses the primary coding requirement from issue #9128: adding pointer-events: none CSS to the ObjectBreadcrumbs component to prevent it from blocking clicks on underlying elements.
Out of Scope Changes check ✅ Passed The PR contains only the necessary pointer-events fix to ObjectBreadcrumbs plus documentation updates (release notes entries) directly related to this bug fix; no unrelated changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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

@dpage dpage force-pushed the fix-9128-breadcrumbs-clicks branch 2 times, most recently from 5e168b0 to 6669c8a Compare June 9, 2026 11:36
The breadcrumbs popup is an absolutely-positioned, informational overlay
at the bottom-left of the object explorer, so it intercepted pointer
events and blocked clicks on the tree items beneath it. Set
pointer-events: none so clicks pass through to the tree.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dpage dpage force-pushed the fix-9128-breadcrumbs-clicks branch from 6669c8a to 2c593e3 Compare June 9, 2026 11:37
@asheshv asheshv requested a review from Copilot June 10, 2026 14:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adjusts the Object Breadcrumbs informational overlay so it no longer intercepts mouse clicks on underlying Object Explorer tree items (fix for #9128), and documents the fix in the 9.16 release notes.

Changes:

  • Set pointerEvents: 'none' on the ObjectBreadcrumbs popup container to allow click-through to tree items beneath.
  • Add a 9.16 release note entry referencing Issue #9128.

Reviewed changes

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

File Description
web/pgadmin/static/js/components/ObjectBreadcrumbs.jsx Makes the breadcrumbs overlay non-interactive so it can’t block underlying clicks.
docs/en_US/release_notes_9_16.rst Adds a bug-fix release note entry for Issue #9128.

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

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.

Bug with interaction of server list

2 participants