Jennifer Morrow [:Boriss] (UX) Reporter
	Description • 13 years ago

In the current iteration of New Tab Page, the background of the thumbnails’ titles is black at 80% opacity.  On colorful and dark thumbnails, this is not very visually distracting and blends nicely into the thumbnails.  However, on mostly-white thumbnails, the 80% black bars appear create a stark horizontal stripe pattern.  By reducing the opacity on those black bars from 80% to 50%, the readability on the dark thumbnails isn’t very affected and the white thumbnails look better because there’s less of a stark contrast between the thumbnails and bars.

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 1 • 13 years ago

Trivial patch.

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 2 • 13 years ago

Asking for ux-review on this change. This patch is also included in the current UX build. Please note that this does not change the black bar at the top - this is gonna be tackled in bug 716532.

	Alex Limi (:limi) — Firefox UX Team
	Comment 3 • 13 years ago

Comment on attachment 590071 [details]
Screenshot showing the new opacity value

This value works for me, just make sure the close/pin on-hover use the same value 

(I know that we're changing these to not use a bar too, but in the meantime it's probably a trivial change to include?)

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 4 • 13 years ago

(In reply to Alex Limi (:limi) — Firefox UX Team from comment #3)
> This value works for me, just make sure the close/pin on-hover use the same
> value 

Fixed.

	Dão Gottwald [:dao]
	Comment 5 • 13 years ago

Comment on attachment 591439 [details] [diff] [review]
patch v2

The whole page would probably be less distracting if this was black text on white with some transparency and maybe a faint border...

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 6 • 13 years ago

Sorry, have to ask for review again. The border color did not match the upper strip's new color. I removed the border completely because we don't need it.

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 7 • 13 years ago

https://hg.mozilla.org/integration/fx-team/rev/bff640abf40c

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 8 • 13 years ago

Merged:

https://hg.mozilla.org/mozilla-central/rev/bff640abf40c

and backed out:

https://hg.mozilla.org/mozilla-central/rev/bc1dcb97c141

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 9 • 13 years ago

https://hg.mozilla.org/integration/fx-team/rev/3ef01a0874d1

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 10 • 13 years ago

https://hg.mozilla.org/mozilla-central/rev/3ef01a0874d1