Closed
Bug 971825
Opened 9 years ago
Closed 9 years ago
"Undo" link after restoring defaults should be a button
Categories
(Firefox :: Toolbars and Customization, defect)
Firefox
Toolbars and Customization
Tracking
()
VERIFIED
FIXED
Firefox 30
People
(Reporter: dao, Assigned: jaws)
References
(Blocks 1 open bug)
Details
(Whiteboard: [Australis:P3])
Attachments
(2 files)
217.91 KB,
image/png
|
shorlander
:
ui-review+
|
Details |
3.04 KB,
patch
|
dao
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Reasoning in bug 970511 comment 18. (In reply to Jared Wein [:jaws] from comment #19) > It's consistency with the New Tab Page thumbnails. That's not reason enough for this to be a link. For one, as I've explained, there are places where we've randomly used links instead of buttons for no reason, and we don't want new UI to be consistent with a mess we've created elsewhere. Secondly, there may actually be a valid reason for the New Tab page to use a link, like the fact that it's inside a box along with the "Tumbnail removed" text where a button may look strange, but that reason won't necessarily translate to about:customizing.
Reporter | ||
Comment 1•9 years ago
|
||
Jared, please take care of this by either making Undo a button or confirming that there's a good reason for it to be a link besides consistency with other unrelated UI. Thanks.
Assignee: nobody → jaws
Assignee | ||
Comment 2•9 years ago
|
||
Stephen, this is what it would look like with a button instead of the Undo link. What do you think?
Attachment #8375122 -
Flags: ui-review?(shorlander)
Comment 3•9 years ago
|
||
I agree that a button is the proper affordance to use here. We could have a whole debate about the use of links for secondary commands or for things with minor consequences, but this probably isn't the place ;) Thanks!
Updated•9 years ago
|
Attachment #8375122 -
Flags: ui-review?(shorlander) → ui-review+
Assignee | ||
Comment 4•9 years ago
|
||
Attachment #8375673 -
Flags: review?(dao)
Assignee | ||
Updated•9 years ago
|
Status: NEW → ASSIGNED
Reporter | ||
Updated•9 years ago
|
Attachment #8375673 -
Flags: review?(dao) → review+
Assignee | ||
Comment 5•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/8e9dfd4baac1
Assignee | ||
Comment 6•9 years ago
|
||
Comment on attachment 8375673 [details] [diff] [review] Patch [Approval Request Comment] Bug caused by (feature/regressing bug #): change in design req. for bug 970511 User impact if declined: change in design between 29 and 30 Testing completed (on m-c, etc.): local, simple change Risk to taking this patch (and alternatives if risky): none String or IDL/UUID changes made by this patch: none
Attachment #8375673 -
Flags: approval-mozilla-aurora?
Comment 7•9 years ago
|
||
I will approve it once it reached m-c.
Comment 8•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8e9dfd4baac1
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
Updated•9 years ago
|
Attachment #8375673 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•9 years ago
|
Whiteboard: [Australis:P3]
Comment 9•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/c9996aa6bdae
status-firefox29:
--- → fixed
status-firefox30:
--- → fixed
Updated•9 years ago
|
QA Contact: cornel.ionce
Comment 10•9 years ago
|
||
Verified as fixed on Windows 7 64 bit, Ubuntu 12.04 and Mac OS X 10.9 using latest Nightly (build ID: 20140313030202) and latest Aurora (build ID: 20140313004000). "Undo" is now displayed as a button instead of a link.
You need to log in
before you can comment on or make changes to this bug.
Description
•