Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Make alignment for tabTitle consistent across OSes #7316

Merged
merged 1 commit into from
Feb 21, 2017
Merged

Make alignment for tabTitle consistent across OSes #7316

merged 1 commit into from
Feb 21, 2017

Conversation

cezaraugusto
Copy link
Contributor

Auditors: @luixxiul

/cc @NejcZdovc @bsclifton for review

Fix #7304
Fix #7312

Main issue is that we had title's height defined and no line-height declared. Removing the former and adding an unitless line-height solved the issue and let us keep it consistent across OSes.

Test Plan:

  1. Tab title should keep vertically centered when website's title has tall characters such as https://github.com/vadimdemedes/trevor
  2. Tab title should keep vertically centered on Windows/Linux distros/Mac
  3. Tab title should keep vertically centered and not cropped if DPI is higher than 100%

@luixxiul
Copy link
Contributor

LGTM

Copy link
Contributor

@NejcZdovc NejcZdovc left a comment

Choose a reason for hiding this comment

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

++ tested it on macOS

@bbondy bbondy merged commit eca0ddc into brave:master Feb 21, 2017
@bsclifton
Copy link
Member

++

@cezaraugusto cezaraugusto deleted the hotfix/tabsbar/7304-7312 branch July 25, 2017 07:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants