[New Tab Page] Remove "site strip" at the top and re-style buttons

RESOLVED FIXED

Status

()

Firefox
Tabbed Browser
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: ttaubert, Assigned: ttaubert)

Tracking

({uiwanted})

Trunk
uiwanted
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Assignee)

Description

6 years ago
The black bar at the top that contains the "pin" and "remove" buttons should be removed. The buttons get a slight re-styling so that they can be stand-alone.
(Assignee)

Updated

6 years ago
Blocks: 716538

Comment 1

6 years ago
Agree here.  I believe this was a change Boriss was going to make?
(Assignee)

Comment 2

6 years ago
Yes, Limi told me this so I figured Boriss knows about it :)
Assignee: ttaubert → jboriss
Keywords: uiwanted
(Assignee)

Updated

5 years ago
No longer blocks: 716538
Created attachment 593709 [details]
Mockup: restyled "site strip" buttons

Instead of a bar, let’s show only the two relevant icons in the same semi-transparent black at the bottom of the thumbnails.  The attached icons are all black, but at 50% opacity on thumbnail mouseover, 70% opacity on icon mouseover, and 90% opacity on click.  When the pin is activated, it should remain in 70% opacity.

It’s true that on some dark pages it will be harder to see the shapes.  But the alternative is a multi-colored icon which looks unlike the overall style of the page or a way to determine the darkness of the thumbnail and adjust accordingly - an option both more complicated and less consistent.  Considering the icons are the same on every thumbnail, simply having the 50% black opacity takes care of most cases and certainly doesn’t hamper a user who uses the icons and thus knows they are there.
Created attachment 593710 [details]
Icons: Three states for pin and X (thumbnail mouseover, icon mouseover, click)
Assignee: jboriss → ttaubert
(Assignee)

Updated

5 years ago
Blocks: 719675
(Assignee)

Comment 5

5 years ago
Created attachment 594156 [details] [diff] [review]
patch v1
(Assignee)

Comment 6

5 years ago
Comment on attachment 594156 [details] [diff] [review]
patch v1

I figured that flagging a screenshot for review doesn't really make sense here. This patch is contained in the latest UX nightlies and waits for ux-review!
Attachment #594156 - Flags: ui-review?(ux-review)
On the latest UX build, the default, pinned and hovered states do not look very different. Also after pinning/unpinning a thumbnail and moving the mouse away, the pin icon does not disappear until another thumbnail is clicked. Is this by design?
Comment on attachment 594156 [details] [diff] [review]
patch v1

Review of attachment 594156 [details] [diff] [review]:
-----------------------------------------------------------------

The difference between hovered states is not sufficient. How about we put a dark gray box around the elements when hovered, and keep the box when pinned? (sorry if this isn't a clear explanation, I'll talk to Boriss about it. :)
Attachment #594156 - Flags: ui-review?(ux-review) → ui-review-
(Assignee)

Comment 9

5 years ago
Waiting for new mockups and/or style directives.
Keywords: uiwanted
Please remove this from UX until an update is ready. (See reasons in comment 7 and comment 8)
(Assignee)

Updated

5 years ago
No longer blocks: 719675
(Assignee)

Comment 11

5 years ago
Fixed by bug 729878.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.