Closed
Bug 344418
Opened 19 years ago
Closed 17 years ago
preferencesCmdUnix.label lacks 3 dots
Categories
(Firefox :: Menus, defect)
Tracking
()
RESOLVED
INVALID
People
(Reporter: john.s.thomsen, Unassigned)
References
()
Details
Attachments
(1 file, 1 obsolete file)
|
1.72 KB,
patch
|
mconnor
:
review-
beltzner
:
ui-review-
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1b1) Gecko/20060708 Firefox/2.0b1
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1b1) Gecko/20060708 Firefox/2.0b1
"Preferences" should be "Preferences..." because a window is opened.
Reproducible: Always
Actual Results:
Preferences
Expected Results:
Preferences...
Comment 1•18 years ago
|
||
Reporter, do you still see this problem with the latest Firefox 2? If not, can you please close this bug as WORKSFORME. Thanks!
Whiteboard: CLOSEME 06/27
Version: unspecified → 2.0 Branch
Yes, I still see it.
I have updated the URL.
Currently the source is:
<!ENTITY preferencesCmd.label "Options...">
<!ENTITY preferencesCmdUnix.label "Preferences">
It should be changed to:
<!ENTITY preferencesCmd.label "Options...">
<!ENTITY preferencesCmdUnix.label "Preferences...">
That is: 3 dots should be added.
This bug has zero priority for 2.0 Branch. I therefore changed the version to trunk. The URL given by me also point to trunk source - if not, please correct me.
Version: 2.0 Branch → Trunk
Updated•18 years ago
|
Whiteboard: CLOSEME 06/27
Comment 3•17 years ago
|
||
CONFIRMED.
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b3pre) Gecko/2008012104 Minefield/3.0b3pre
Updated•17 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 4•17 years ago
|
||
This adds ellipsis to preferencesCmdUnix.label. Does this need entity name change?
Attachment #298317 -
Flags: review?(mano)
While I personally want the ellipsis (and have them thanks to my own extension), I noted a week ago, this comment:
https://bugzilla.mozilla.org/show_bug.cgi?id=412387#c14
which invalidate my argument for having them. Following the bug mentioned there, I read another comment:
https://bugzilla.mozilla.org/show_bug.cgi?id=255051#c5
Linking to a GNOME style guide, saying:
"Label the menu item with a trailing ellipsis ("...") only if the command requires further input from the user before it can be performed. Do not add an ellipsis to items that only present a confirmation dialog (such as Delete), or that do not require further input (such as Properties, Preferences or About)."
Since Firefox for Linux is turning into a genuine looking GNOME app these days, I believe the lack of dots may be okay.
Comment 6•17 years ago
|
||
Comment on attachment 298317 [details] [diff] [review]
add ellipsis to preferencesCmdUnix.label
So, yeah, this needs careful ui-r. You should also rev the entity name.
Attachment #298317 -
Flags: review?(mano)
Comment 7•17 years ago
|
||
Attachment #298317 -
Attachment is obsolete: true
Attachment #298963 -
Flags: ui-review?(beltzner)
Attachment #298963 -
Flags: review?(mano)
Comment 8•17 years ago
|
||
Comment on attachment 298963 [details] [diff] [review]
v2
On GNOME this should stay without an ellipsis.
Attachment #298963 -
Flags: ui-review?(beltzner) → ui-review-
Updated•17 years ago
|
Attachment #298963 -
Flags: review?(mano) → review-
Comment 9•17 years ago
|
||
GNOME HIG is explicit. http://developer.gnome.org/projects/gup/hig/1.0/menus.html
So is Windows UIG: http://msdn2.microsoft.com/en-us/library/aa511502.aspx#ellipses
INVALID.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•