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

Align resolve button text and size icon appropriately #10646

Merged
merged 1 commit into from Sep 25, 2020
Merged

Conversation

niik
Copy link
Member

@niik niik commented Sep 25, 2020

Description

In #10224 we upgraded the version of octicons that we ship and this little fella missed our careful audit of changes. We had an explicit height and width set for the arrow down octicon that didn't align with the dimensions of the new icon causing the text in the button to be misaligned.

Screenshots

Before

Screen Shot 2020-09-25 at 14 23 11

After

Screen Shot 2020-09-25 at 14 22 59

Release notes

Notes: no-notes

@niik niik requested a review from rafeca September 25, 2020 12:27
Copy link
Contributor

@rafeca rafeca left a comment

Choose a reason for hiding this comment

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

LGTM! ✨

@niik niik merged commit f623089 into development Sep 25, 2020
@niik niik deleted the button-tweak branch September 25, 2020 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants