Skip to content
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

The dirty indicator does not get cleared up after reverting changes #10812

Merged
merged 2 commits into from Aug 13, 2021

Conversation

fcollonval
Copy link
Member

@fcollonval fcollonval commented Aug 12, 2021

References

Fixes #10431

Code changes

The initial state of executedCode is set to match the cell input if and only if the execution count saved in the notebook is not null.

indicator

User-facing changes

The dirty indicator will be triggered for executed cell without outputs.

Backwards-incompatible changes

N/A

@jupyterlab-dev-mode
Copy link

Thanks for making a pull request to JupyterLab!

To try out this branch on binder, follow this link: Binder

@fcollonval fcollonval self-assigned this Aug 12, 2021
@fcollonval fcollonval added the bug label Aug 12, 2021
@fcollonval fcollonval added this to the 4.0 milestone Aug 12, 2021
@fcollonval fcollonval modified the milestones: 4.0, 3.1.x Aug 12, 2021
@fcollonval fcollonval marked this pull request as ready for review August 12, 2021 09:19
@blink1073
Copy link
Member

Kicking CI

@blink1073 blink1073 closed this Aug 12, 2021
@blink1073 blink1073 reopened this Aug 12, 2021
Copy link
Member

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@fcollonval fcollonval merged commit 292d43b into jupyterlab:master Aug 13, 2021
@fcollonval fcollonval deleted the fcollonval/issue10431 branch August 13, 2021 14:39
@fcollonval
Copy link
Member Author

@meeseeksdev please backport to 3.1.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Aug 14, 2021
blink1073 pushed a commit that referenced this pull request Aug 14, 2021
… reverting changes (#10833)

Co-authored-by: Frédéric Collonval <fcollonval@gmail.com>
@github-actions github-actions bot added the status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label Feb 11, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug pkg:cells status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The dirty indicator does not get cleared up after reverting changes
3 participants