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
Fixup highlighted graph #11123
Fixup highlighted graph #11123
Conversation
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.
+1
Looks good, have not run
} | ||
set { _revisionGraphDrawStyle = value; } | ||
} | ||
public RevisionGraphDrawStyle RevisionGraphDrawStyle; |
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.
This makes it a public field, which is discouraged.
public RevisionGraphDrawStyle RevisionGraphDrawStyle; | |
public RevisionGraphDrawStyle RevisionGraphDrawStyle { get; set; } |
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.
done
only if unrelated branches are colored addendum to gitextensions#10915
c8a617c
to
acb4b84
Compare
only if unrelated branches are colored addendum to gitextensions#10915
acb4b84
to
8138a08
Compare
Fixes #11098
regressed in #10915
Proposed changes
RevisionGraphColumnProvider.RevisionGraphDrawStyle
Screenshots
Before
After
Test methodology
Please do not squash merge
✒️ I contribute this code under The Developer Certificate of Origin.