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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve file-finder-buffer style #3571

Merged
merged 3 commits into from
Sep 19, 2020

Conversation

kidonng
Copy link
Member

@kidonng kidonng commented Sep 17, 2020

Thanks for contributing! 馃崉

  1. LINKED ISSUES:
    Close file-finder-buffer鈥檚 field is now white聽#3212

  2. TEST URLS:
    https://github.com/sindresorhus/refined-github

  3. SCREENSHOT:
    Before:
    After:
    image

@fregante fregante added the bug label Sep 17, 2020
Comment on lines 11 to 12
className="form-control tree-finder-input p-0 border-0"
style={{marginTop: '-0.35em', backgroundColor: 'transparent'}}
Copy link
Member

Choose a reason for hiding this comment

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

Alignment was not an issue so don't change that.

Suggested change
className="form-control tree-finder-input p-0 border-0"
style={{marginTop: '-0.35em', backgroundColor: 'transparent'}}
className="form-control tree-finder-input p-0 border-0"
style={{marginTop: '-0.19em', backgroundColor: 'transparent'}}

This is now misaligned:

Copy link
Member Author

Choose a reason for hiding this comment

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

0.35em is the right alignment for me. Could you please try 0.19em with vertical-align: baseline?

@fregante fregante merged commit 0b7e946 into refined-github:master Sep 19, 2020
@kidonng kidonng deleted the file-finder-buffer branch October 8, 2020 00:40
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.

file-finder-buffer鈥檚 field is now white
3 participants