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 handling of code editor refresh #7672
Fix handling of code editor refresh #7672
Conversation
Thanks for making a pull request to JupyterLab! To try out this branch on binder, follow this link: |
packages/codeeditor/src/widget.ts
Outdated
@@ -141,7 +141,7 @@ export class CodeEditorWrapper extends Widget { | |||
* A message handler invoked on an `'after-show'` message. | |||
*/ | |||
protected onAfterShow(msg: Message): void { | |||
if (this._updateOnShow) { | |||
if (this._updateOnShow || !this._hasRefreshed) { |
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.
It seems like this should be &&
, not ||
? Also, should _hasRefreshed
be reset to false on hide?
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.
I updated the variable name and added a comment, as well as a reset whenever we attach.
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.
Thanks. That name change and comment made the logic clearer. I'll test.
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.
Thanks!
👍
References
Fixes #7671
Code changes
Handles the case where a code editor is not visible when it tries to update. This can happen when the widget is attached and visible, and then is no longer visible when
onUpdateRequest
is later called.User-facing changes
None in core JupyterLab, affects extension authors using code editors and things that depend on them (like notebooks).
Backwards-incompatible changes
None.