Skip to content

Don't show custom context menu on right-click in Monaco editor#202

Merged
vbabich merged 5 commits intodeephaven:mainfrom
vbabich:context-menu-fix
Sep 27, 2021
Merged

Don't show custom context menu on right-click in Monaco editor#202
vbabich merged 5 commits intodeephaven:mainfrom
vbabich:context-menu-fix

Conversation

@vbabich
Copy link
Collaborator

@vbabich vbabich commented Sep 17, 2021

Fix the issue with an unusable Monaco context menu when Monaco is nested in a component having its own ContextMenu with a non-empty actions list.

@vbabich vbabich added bug Something isn't working web-client-ui labels Sep 17, 2021
@vbabich vbabich requested a review from mofojed September 17, 2021 19:45
@vbabich vbabich self-assigned this Sep 17, 2021
@vbabich vbabich requested a review from mofojed September 17, 2021 21:57
mofojed
mofojed previously approved these changes Sep 20, 2021
@vbabich vbabich requested a review from mofojed September 24, 2021 20:34
@vbabich vbabich merged commit 34e0980 into deephaven:main Sep 27, 2021
mofojed added a commit to mofojed/web-client-ui that referenced this pull request Oct 12, 2021
* `chart`, `code-studio`, `components`, `console`, `dashboard-core-plugins`, `dashboard`, `file-explorer`, `golden-layout`, `grid`, `iris-grid`, `jsapi-shim`, `log`, `mocks`, `react-hooks`, `redux`, `storage`, `utils`
  * [deephaven#203](deephaven#203) Separate dashboard into own package ([@mofojed](https://github.com/mofojed))

* `chart`, `code-studio`, `components`, `console`, `dashboard-core-plugins`, `dashboard`, `file-explorer`, `golden-layout`, `grid`, `iris-grid`, `jsapi-shim`, `log`, `mocks`, `react-hooks`, `redux`, `storage`, `utils`
  * [deephaven#203](deephaven#203) Separate dashboard into own package ([@mofojed](https://github.com/mofojed))
* `icons`
  * [deephaven#220](deephaven#220) update to latest @vscode/codicons and add dhFreeze icon ([@dsmmcken](https://github.com/dsmmcken))

* `code-studio`
  * [deephaven#225](deephaven#225) Handle invalid layout files from the server correctly ([@mofojed](https://github.com/mofojed))
* `components`, `console`
  * [deephaven#202](deephaven#202) Don't show custom context menu on right-click in Monaco editor ([@vbabich](https://github.com/vbabich))

* [deephaven#226](deephaven#226) Add fetch-depth flag in alpha publish ([@mofojed](https://github.com/mofojed))
* [deephaven#224](deephaven#224) Update publish action auth ([@mofojed](https://github.com/mofojed))

- Don ([@dsmmcken](https://github.com/dsmmcken))
- Mike Bender ([@mofojed](https://github.com/mofojed))
- [@vbabich](https://github.com/vbabich)
@mofojed mofojed mentioned this pull request Oct 12, 2021
mofojed added a commit that referenced this pull request Oct 12, 2021
* `chart`, `code-studio`, `components`, `console`, `dashboard-core-plugins`, `dashboard`, `file-explorer`, `golden-layout`, `grid`, `iris-grid`, `jsapi-shim`, `log`, `mocks`, `react-hooks`, `redux`, `storage`, `utils`
  * [#203](#203) Separate dashboard into own package ([@mofojed](https://github.com/mofojed))

* `chart`, `code-studio`, `components`, `console`, `dashboard-core-plugins`, `dashboard`, `file-explorer`, `golden-layout`, `grid`, `iris-grid`, `jsapi-shim`, `log`, `mocks`, `react-hooks`, `redux`, `storage`, `utils`
  * [#203](#203) Separate dashboard into own package ([@mofojed](https://github.com/mofojed))
* `icons`
  * [#220](#220) update to latest @vscode/codicons and add dhFreeze icon ([@dsmmcken](https://github.com/dsmmcken))

* `code-studio`
  * [#225](#225) Handle invalid layout files from the server correctly ([@mofojed](https://github.com/mofojed))
* `components`, `console`
  * [#202](#202) Don't show custom context menu on right-click in Monaco editor ([@vbabich](https://github.com/vbabich))

* [#226](#226) Add fetch-depth flag in alpha publish ([@mofojed](https://github.com/mofojed))
* [#224](#224) Update publish action auth ([@mofojed](https://github.com/mofojed))

- Don ([@dsmmcken](https://github.com/dsmmcken))
- Mike Bender ([@mofojed](https://github.com/mofojed))
- [@vbabich](https://github.com/vbabich)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working web-client-ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants