-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix stackplots static style #5045
Conversation
* added the ability to pass the element you would like to enlarge to the view large action * Example of performance marks (#5027) Co-authored-by: John Hill <john.c.hill@nasa.gov> Co-authored-by: unlikelyzero <jchill2@gmail.com> Co-authored-by: Andrew Henry <andrew.k.henry@nasa.gov>
* adding transactions to notebook entry editing Co-authored-by: Nikhil <nikhil.k.mandlik@nasa.gov> Co-authored-by: Andrew Henry <andrew.k.henry@nasa.gov>
… prevents any ambiguity about which element should receive the style
Codecov Report
@@ Coverage Diff @@
## master #5045 +/- ##
==========================================
+ Coverage 49.84% 49.90% +0.05%
==========================================
Files 556 534 -22
Lines 20393 19549 -844
Branches 1876 1767 -109
==========================================
- Hits 10164 9755 -409
+ Misses 9736 9348 -388
+ Partials 493 446 -47
Continue to review full report at Codecov.
|
sorry @shefalijoshi , i think i let this get stale and now its out of sync with master |
…ckplots-static-style
@nikhilmandlik , can you take a look at this one? |
* @private | ||
*/ | ||
_onCancel() { | ||
//noop |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ahh thanks,
there are more like this tracked here, #4864
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very small change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Needs testing fix |
Closes #4912 #4864
Describe your changes:
The StyleRuleManager was subscribing to condition set updates even when it was destroyed (via a promise). Now we check to make sure that the StyleRuleManager is not destroyed when we handle the promise.
Cancelling the properties dialog was not handled. Added a handler for this.
All Submissions:
Author Checklist
Reviewer Checklist