-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
ensure conflicts popup and banner are cleaned up if conflicts no longer exist #7209
Conversation
LGTM. Ran both |
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.
Love it
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.
Love it even more 📖
@niik no, i love it more! |
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.
my love for this is OFF THE CHARTS
Overview
Closes #7046
Description
This PR adds a
AppStore.clearConflictsFlowVisuals()
function to ensure any UI related to conflicts (both merge conflicts and rebase conflicts) goes away if the rebase/merge is aborted outside the app.It wasn't clear to me from the original report how deep to go with cleaning up state, but we can iterate on this based on further testing and feedback.
Release notes
Notes: no-notes