Last Comment Bug 721087 - [New Tab Page] Cells should have an outline to indicate their positions
: [New Tab Page] Cells should have an outline to indicate their positions
Status: RESOLVED FIXED
:
Product: Firefox
Classification: Client Software
Component: Tabbed Browser (show other bugs)
: Trunk
: All All
: -- normal (vote)
: Firefox 12
Assigned To: Tim Taubert [:ttaubert]
:
: Dão Gottwald [:dao]
Mentors:
Depends on:
Blocks: 716538
  Show dependency treegraph
 
Reported: 2012-01-25 09:36 PST by Tim Taubert [:ttaubert]
Modified: 2012-01-31 01:26 PST (History)
5 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Screenshot of the current patch (116.59 KB, image/png)
2012-01-25 09:39 PST, Tim Taubert [:ttaubert]
limi: ui‑review+
Details
patch v1 (1.59 KB, patch)
2012-01-25 09:40 PST, Tim Taubert [:ttaubert]
dao+bmo: review+
Details | Diff | Splinter Review

Description Tim Taubert [:ttaubert] 2012-01-25 09:36:04 PST
Give cells a dashed outline to indicate their positions when empty or during a drag/drop operation.
Comment 1 Tim Taubert [:ttaubert] 2012-01-25 09:39:15 PST
Created attachment 591506 [details]
Screenshot of the current patch
Comment 2 Tim Taubert [:ttaubert] 2012-01-25 09:40:47 PST
Created attachment 591507 [details] [diff] [review]
patch v1

Adding dashed outlines to cells.
Comment 3 Tim Taubert [:ttaubert] 2012-01-25 15:54:40 PST
Comment on attachment 591506 [details]
Screenshot of the current patch

Got "review" over IRC already but requesting formally for the record here :)
Comment 4 Alex Limi (:limi) — Firefox UX Team 2012-01-30 12:16:57 PST
Comment on attachment 591506 [details]
Screenshot of the current patch

Looks good to me!

Note You need to log in before you can comment on or make changes to this bug.