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

Fix branch dropdown position in clean-conversation-headers #5598

Merged
merged 2 commits into from
Apr 21, 2022

Conversation

cheap-glitch
Copy link
Contributor

Fixes the placement of the arrow icon, relative to the base branch dropdown that appears when editing the title of an open PR.

Test URLs

Screenshot

Before

before

After

after

Comment on lines 50 to 53
/* Fixes vertical alignment of base branch dropdown */
.rgh-clean-conversation-headers .commit-ref-dropdown {
margin-top: -2px !important;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This could technically be part of github-bugs.css since afaict it's not caused by this feature.

Copy link
Member

@yakov116 yakov116 Apr 18, 2022

Choose a reason for hiding this comment

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

The comment should have an issue number though

@fregante
Copy link
Member

fregante commented Apr 19, 2022

🤔

It works well for me in 22.4.1 Safari

Screen Shot

Was it broken by the fix for #5585?

@cheap-glitch
Copy link
Contributor Author

cheap-glitch commented Apr 19, 2022

It works well for me in 22.4.1 Safari

It was likely broken by #5322, which was released in 22.4.11

@cheap-glitch cheap-glitch changed the title Fix handling of base branch dropdown in clean-conversation-headers Fix branch dropdown position in clean-conversation-headers Apr 21, 2022
@cheap-glitch cheap-glitch merged commit 5141515 into main Apr 21, 2022
@cheap-glitch cheap-glitch deleted the handle-branch-dropdown branch April 21, 2022 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants