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.
Agree here. I believe this was a change Boriss was going to make?
Yes, Limi told me this so I figured Boriss knows about it :)
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)
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!
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. :)
Waiting for new mockups and/or style directives.
Fixed by bug 729878.