Skip to content

fix: Save or discard a changed notebook does not close modal on first click#1188

Merged
mattrunyon merged 1 commit intodeephaven:mainfrom
mattrunyon:notebook-close-modal-fix
Mar 30, 2023
Merged

fix: Save or discard a changed notebook does not close modal on first click#1188
mattrunyon merged 1 commit intodeephaven:mainfrom
mattrunyon:notebook-close-modal-fix

Conversation

@mattrunyon
Copy link
Copy Markdown
Collaborator

Fixes #1187

@mattrunyon mattrunyon self-assigned this Mar 30, 2023
@mattrunyon mattrunyon changed the title fix: Save or discard a changed notebook does not close modal fix: Save or discard a changed notebook does not close modal on first click Mar 30, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 30, 2023

Codecov Report

Merging #1188 (d9d08fc) into main (f1f3abf) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##             main    #1188    +/-   ##
========================================
  Coverage   44.19%   44.19%            
========================================
  Files         448      448            
  Lines       33308    33424   +116     
  Branches     8362     8403    +41     
========================================
+ Hits        14719    14771    +52     
- Misses      18540    18603    +63     
- Partials       49       50     +1     
Flag Coverage Δ
unit 44.19% <0.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ashboard-core-plugins/src/panels/NotebookPanel.tsx 1.60% <0.00%> (ø)

... and 4 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@mattrunyon mattrunyon merged commit bba2d01 into deephaven:main Mar 30, 2023
@mattrunyon mattrunyon deleted the notebook-close-modal-fix branch March 30, 2023 16:08
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Save or discard a changed notebook does not close modal on first click

3 participants