Closed
Bug 839054
Opened 12 years ago
Closed 12 years ago
Open containing folder/Show in Finder are both displayed on Mac OS X
Categories
(Firefox :: Downloads Panel, defect)
Tracking
()
RESOLVED
FIXED
Firefox 21
Tracking | Status | |
---|---|---|
firefox20 | --- | verified |
People
(Reporter: sbadau, Assigned: mconley)
Details
(Keywords: regression)
Attachments
(1 file)
3.75 KB,
patch
|
mak
:
review+
Gavin
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Windows NT 6.1; rv:21.0) Gecko/20130206 Firefox/21.0
Build ID: 20130206031027
Steps to reproduce:
1. Launch Firefox
2. Perform several downloads
3. Open the panel or the downloads view and right click on any completed download
4. While in the panel or in the downloads view focus the right most button.
Expected results:
On Mac OS X the button's tooltip and the option from the context menu are the same.
Actual results:
Same action is done when selecting either Show in Finder option or Open Containing Folder button. To avoid confusion, the option and the button's tooltip should have the same name.
Reporter | ||
Comment 1•12 years ago
|
||
Regression range:
Last good nightly: 2013-01-01
First bad nightly: 2013-01-02
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=dec6aa71da64&tochange=a812ef63de87
Reporter | ||
Comment 2•12 years ago
|
||
The issue is also reproducible on the latest Aurora:
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:20.0) Gecko/20130208 Firefox/20.0
Build ID: 20130208042018
Assignee | ||
Comment 4•12 years ago
|
||
This seems to do the job.
Assignee: nobody → mconley
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•12 years ago
|
||
Comment on attachment 711925 [details] [diff] [review]
Patch v1
Bug 746783 was hiding this bug from us, but didn't introduce it. We never displayed the right strings for the icons.
Attachment #711925 -
Flags: review?(mak77)
Comment 6•12 years ago
|
||
Comment on attachment 711925 [details] [diff] [review]
Patch v1
Review of attachment 711925 [details] [diff] [review]:
-----------------------------------------------------------------
oops!
Attachment #711925 -
Flags: review?(mak77) → review+
Assignee | ||
Comment 7•12 years ago
|
||
Comment 8•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 21
Comment 9•12 years ago
|
||
Comment on attachment 711925 [details] [diff] [review]
Patch v1
[Approval Request Comment]
Bug caused by (feature/regressing bug #): downloads panel feature
User impact if declined: wrong string on Mac
Testing completed (on m-c, etc.): m-c
Risk to taking this patch (and alternatives if risky): minor, ifdef using different string on Mac
String or UUID changes made by this patch: none
Attachment #711925 -
Flags: approval-mozilla-aurora?
Reporter | ||
Comment 10•12 years ago
|
||
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:21.0) Gecko/20130212 Firefox/21.0
Build ID: 20130212182646
Verified as fixed on the latest hourly tinderbox mozilla central build: - the button's tooltip and the contextual option from the panel and view is "Show in Finder".
Updated•12 years ago
|
Attachment #711925 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 11•12 years ago
|
||
status-firefox20:
--- → fixed
Reporter | ||
Comment 12•12 years ago
|
||
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:20.0) Gecko/20100101 Firefox/20.0
Build ID: 20130220104816
Verified as fixed on Firefox 20 beta 1 - the button's tooltip and the option in the context menu have the same name which is - Show in Finder.
You need to log in
before you can comment on or make changes to this bug.
Description
•