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

New tab button too high #3208

Closed
D0J0P opened this issue Aug 16, 2016 · 7 comments
Closed

New tab button too high #3208

D0J0P opened this issue Aug 16, 2016 · 7 comments
Labels
design A design change, especially one which needs input from the design team. polish Nice to have — usually related to front-end/visual tasks. QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64
Milestone

Comments

@D0J0P
Copy link

D0J0P commented Aug 16, 2016

I find that the new tab button on the tab bar is just a bit too high. It looks off-center. It would look better if it were centered.

@luixxiul
Copy link
Contributor

#1863, font-family seems to be causing the issue here / cc: @Sh1d0w

@luixxiul
Copy link
Contributor

The perfect solution here should be using a FontAwesome icon like others but there is no proper one, which has been proposed here FortAwesome/Font-Awesome#2161

@bradleyrichter
Copy link
Contributor

This is on the polish list. We will fix for 1.0.

@bradleyrichter bradleyrichter added design A design change, especially one which needs input from the design team. polish Nice to have — usually related to front-end/visual tasks. labels Aug 17, 2016
@bradleyrichter bradleyrichter added this to the 1.0.0 milestone Aug 17, 2016
@D0J0P
Copy link
Author

D0J0P commented Aug 17, 2016

Thank you Bradley. I had no idea there was going to be a 1.0 so soon. Do you know when? I don't follow Brave browser very much, as it's not my primary browser, but I'm interested in it because it's the most lightweight browser I've tried so far.

@neeklamy
Copy link
Contributor

@D0J0P, the Brave Development Timeline says 1.0 will ship 26 August 2016.

@roqstr
Copy link

roqstr commented Aug 17, 2016

@D0J0P @neeklamy I've been told the milestones are more accurate.

@neeklamy
Copy link
Contributor

Thanks, @roqstr, so the 1.0.0 milestone says it’s due on the 9 September.

It’d be very handy if the current version info was made available on the homepage with a link to the upcoming milestones.

bbondy added a commit that referenced this issue Aug 31, 2016
Auditors: @alexwykoff

Test Plan: n/a

General feedback for future release notes:

Major new features at the top: Autofill is a major feature highlight but
wasn’t listed, so should be at the top.  A sub-fix of it was listed
under fixes.

Sub-fixes for a new feature: about:history is a new feature so sub-fixes
related to that  aren’t needed in release notes.  Ditto ledger stuff.

Exclude Tests: Anything that mentions fixing automated tests or tests
shouldn’t be included.  In general anything dev relatd doesn't need to
be included.

Some reformatting of titles needed so that a casual reader can
understand it, remove sub-references to other issues inside titles.

Duplicate bugs: “Fixed new tab button too high” and “Fixed new tab
button position fixes #3208” are too similar so just mention one of them

Simplify some titles: “Fixed Make targets.host UNIQUE, which
auto-creates index” -> “Optimize HTTPS Everywhere”

Remove text like “Like other browsers do” from titles

Consistent naming of main features “non script” “noScript” -> “NoScript”

Always end lines in a period.

Thanks!
bbondy added a commit that referenced this issue Aug 31, 2016
Auditors: @alexwykoff

Test Plan: n/a

General feedback for future release notes:

Major new features at the top: Autofill is a major feature highlight but
wasn’t listed, so should be at the top.  A sub-fix of it was listed
under fixes.

Sub-fixes for a new feature: about:history is a new feature so sub-fixes
related to that  aren’t needed in release notes.  Ditto ledger stuff.

Exclude Tests: Anything that mentions fixing automated tests or tests
shouldn’t be included.  In general anything dev relatd doesn't need to
be included.

Some reformatting of titles needed so that a casual reader can
understand it, remove sub-references to other issues inside titles.

Duplicate bugs: “Fixed new tab button too high” and “Fixed new tab
button position fixes #3208” are too similar so just mention one of them

Simplify some titles: “Fixed Make targets.host UNIQUE, which
auto-creates index” -> “Optimize HTTPS Everywhere”

Remove text like “Like other browsers do” from titles

Consistent naming of main features “non script” “noScript” -> “NoScript”

Always end lines in a period.

Thanks!
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
design A design change, especially one which needs input from the design team. polish Nice to have — usually related to front-end/visual tasks. QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64
Projects
None yet
Development

No branches or pull requests

7 participants