Skip to content

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

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

Loading