Closed
Bug 343909
Opened 19 years ago
Closed 19 years ago
The 'History' menu's access key "i" should be changed (underline way to small)
Categories
(Firefox :: Toolbars and Customization, defect)
Firefox
Toolbars and Customization
Tracking
()
VERIFIED
FIXED
People
(Reporter: u88484, Assigned: u88484)
Details
(Keywords: access, verified1.8.1)
Attachments
(2 files, 1 obsolete file)
1.50 KB,
image/png
|
Details | |
1.23 KB,
patch
|
asaf
:
review+
beltzner
:
ui-review+
beltzner
:
approval1.8.1+
|
Details | Diff | Splinter Review |
Summary: The istory access key 'i' → The 'History' menu's access key "i" should be changed (underline way to small)
Patch which changes the access key from 'i' to 'y'...if this of course decided.
Attachment #228474 -
Flags: ui-review?(beltzner)
Comment 3•19 years ago
|
||
Comment on attachment 228474 [details] [diff] [review]
patch
I think "s" works better than "y".
Attachment #228474 -
Flags: ui-review?(beltzner) → ui-review-
Patch per comment 3, changes access key to "s"
Attachment #228474 -
Attachment is obsolete: true
Attachment #230105 -
Flags: ui-review?(beltzner)
Comment 5•19 years ago
|
||
Comment on attachment 230105 [details] [diff] [review]
Patch
Thanks, Kurt. Do we need to rev the entity names for localizers? I don't think so, since they'll have xlated it ...
Attachment #230105 -
Flags: ui-review?(beltzner)
Attachment #230105 -
Flags: ui-review+
Attachment #230105 -
Flags: review?(mconnor)
Comment 6•19 years ago
|
||
Comment on attachment 230105 [details] [diff] [review]
Patch
This is a bug specific to the en-US package so no need to rev the entity name, r=mano.
Attachment #230105 -
Flags: review?(mconnor) → review+
Comment on attachment 230105 [details] [diff] [review]
Patch
Trivial, no risk fix
Attachment #230105 -
Flags: approval1.8.1?
Comment 8•19 years ago
|
||
s is definitely better than y. So this is good.
FYI, we actually have a FAQ on choosing accesskeys. Avoid 1 pixel thin lowercase letters like i or l in certain fonts. Avoid letters with descenders like y, q, g. It makes it too hard to see the underline.
More info here:
http://www.mozilla.org/access/keyboard/accesskey
Comment 9•19 years ago
|
||
mozilla/browser/locales/en-US/chrome/browser/browser.dtd 1.52
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
Comment 10•19 years ago
|
||
Comment on attachment 230105 [details] [diff] [review]
Patch
a=beltzner, single hand saluting this one as per an irc convo with connor last night.
Attachment #230105 -
Flags: approval1.8.1? → approval1.8.1+
Updated•19 years ago
|
Whiteboard: [Branch checkin needed] → [checkin needed (1.8 branch)]
Comment 11•19 years ago
|
||
mozilla/browser/locales/en-US/chrome/browser/browser.dtd 1.25.2.23
Keywords: fixed1.8.1
Whiteboard: [checkin needed (1.8 branch)]
Comment 12•19 years ago
|
||
I'm fond of the shortcut Alt+S
to submit my replies in most forums.
But now I found the shortcut has been occupied by fx for the bookmarks shortcut.
I wish that the staff can cancel this shortcut.
Please, Alt+S to submit is really vital for many forums users.
Assignee | ||
Comment 13•18 years ago
|
||
Mike, there are two threads on mozillazine complaining about alt+s being used or was used a lot to submit on forms and forums. Think we need to change the access key to something else, suggestions?
http://forums.mozillazine.org/viewtopic.php?t=446830
http://forums.mozillazine.org/viewtopic.php?t=452480
Comment 14•18 years ago
|
||
(In reply to comment #13)
> alt+s being used or was used a lot to submit on forms and forums.
That's because of bug 348793 which changed that shortcut from Alt+S to Alt+Shift+S and has nothing to do with this bug (users who prefer Alt+S can get the former behavior by reverting ui.key.generalAccessKey to 18; then content accesskeys will once again overrule chrome ones - which was the main reason for the change).
Status: RESOLVED → VERIFIED
Keywords: verified1.8.1
Comment 15•18 years ago
|
||
Can't say I'm a big fan of the new change, pretty much all forum software (vBulletin, IPB, phpBB, SMF, YaBB etc) uses alt+s as the shortcut for sending a post.
This change will annoy quite a few people, suggesting people change stuff in their theme won't help none techy users but probably force them to move to another browser instead.
Comment 16•18 years ago
|
||
(In reply to comment #15)
> Can't say I'm a big fan of the new change, pretty much all forum software
> (vBulletin, IPB, phpBB, SMF, YaBB etc) uses alt+s as the shortcut for sending a
> post.
>
> This change will annoy quite a few people, suggesting people change stuff in
> their theme won't help none techy users but probably force them to move to
> another browser instead.
>
I agree. I am very much involved with Invision, and on Invision forums for most of my day. I was sooooooooooooooooo tempted to revert to the past version after the beta upgrade because the alt+s kept opening the history. To find the about:config fix, I had to search mozillazine for the keyword "accesskey", which non-html type people will most likely not search for, because the will not know what it is or what it does.
In all due respect, the people that will gripe about this are probably the ones that know how to research out the fix, but why should we? A web browser should be a tool to utilize a person's website, not override the designer/coder's intentions.
Comment 17•18 years ago
|
||
(In reply to comment #15) and (In reply to comment #16)
As you could have read in comment #14, this bug is not to blame for your issues. In case you do have constructive feedback, please leave it in bug 349716 - otherwise please use the MozillaZine forums for venting.
Updated•18 years ago
|
Keywords: fixed1.8.1
Comment 18•18 years ago
|
||
I just wanna throw my vote in. I use ALT+S for submitting, lots of places! I never use history :P Not having it work is frustrating. I am coming to hate the history menu :P
Comment 19•18 years ago
|
||
Comment 18 is irrelevant here. Please see comment 17.
You need to log in
before you can comment on or make changes to this bug.
Description
•