Skip to content

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

Merged
cheap-glitch merged 2 commits into
mainfrom
handle-branch-dropdown
Apr 21, 2022
Merged

Fix branch dropdown position in clean-conversation-headers#5598
cheap-glitch merged 2 commits into
mainfrom
handle-branch-dropdown

Conversation

@cheap-glitch

Copy link
Copy Markdown
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
Copy Markdown
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.

@yakov116 yakov116 Apr 18, 2022

Copy link
Copy Markdown
Member

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

fregante commented Apr 19, 2022

Copy link
Copy Markdown
Member

🤔

It works well for me in 22.4.1 Safari

Screen Shot

Was it broken by the fix for #5585?

@cheap-glitch

cheap-glitch commented Apr 19, 2022

Copy link
Copy Markdown
Contributor Author

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