-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
Restore swap-branches-on-compare
#6457
Conversation
swap-branches-on-compare
swap-branches-on-compare
Yeah a button works too, in that case there's no need to hide 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.
LGTM, I'm just afraid that it steals too much attention with a regular button style. Don't like btn-octicon
?
Because it is... not an icon? Looks too weird and not something clickable to me. |
Hey, thanks! A few times, before this feature reappeared, I clicked the three dots beneath the arrow (between the two things to be compared), imagining that a menu would appear with Swap amongst its options. Oops. |
The three dots link is a nice touch nonetheless, it just happened to break our feature. |
Fix #6361
Test URLs
23.2.1...main
Screenshot