Skip to content
Snippets Groups Projects

JetBrains: Force hide the material UI overlay when the Sourcegraph window is hidden

Merged Warren Gifford requested to merge ps/jetbrains-material-ui-overlay-fix-two into main

Created by: philipp-spiess

Closes #36479 (closed)

Another attempt at fixing #36479 (closed). This time it does not bring back the web view issues on macOS.

We now manually emit an action defined by the material UI theme to hide the overlay it opens whenever a popover is created. This third-party plugin does not work with our approach of keeping the popover alive and thus, when the Sourcegraph popover is closed, their custom overlay stays active.

Background on how we came up with the solution can be found here: https://github.com/mallowigi/material-theme-issues/issues/179#issuecomment-1151169850

Note: This is not an ideal fix yet as the overlay does not re-appear when the window is re-opened. Given that this is not the only issue with the overlay and the fact that I was able to find more by using this theme myself for a bit, I am not super worried about this.

Test plan

https://user-images.githubusercontent.com/458591/172866469-3e0af482-1e5e-499c-b720-5394974a4c80.mov

App preview:

Check out the client app preview documentation to learn more.

Merge request reports

Approval is optional

Merged by avatar (May 5, 2025 9:07pm UTC)

Merge details

  • Changes merged into main with 750b9ddb.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading