Closed Bug 343909 Opened 15 years ago Closed 15 years ago

The 'History' menu's access key "i" should be changed (underline way to small)

Categories

(Firefox :: Toolbars and Customization, defect)

defect
Not set
trivial

Tracking

()

VERIFIED FIXED

People

(Reporter: u88484, Assigned: u88484)

Details

(Keywords: access, verified1.8.1)

Attachments

(2 files, 1 obsolete file)

 
Summary: The istory access key 'i' → The 'History' menu's access key "i" should be changed (underline way to small)
Assignee: nobody → supernova_00
Status: NEW → ASSIGNED
Attached patch patch (obsolete) — Splinter Review
Patch which changes the access key from 'i' to 'y'...if this of course decided.
Attachment #228474 - Flags: ui-review?(beltzner)
Keywords: access
Comment on attachment 228474 [details] [diff] [review]
patch

I think "s" works better than "y".
Attachment #228474 - Flags: ui-review?(beltzner) → ui-review-
Attached patch PatchSplinter Review
Patch per comment 3, changes access key to "s"
Attachment #228474 - Attachment is obsolete: true
Attachment #230105 - Flags: ui-review?(beltzner)
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 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+
Whiteboard: [checkin needed]
Comment on attachment 230105 [details] [diff] [review]
Patch

Trivial, no risk fix
Attachment #230105 - Flags: approval1.8.1?
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
mozilla/browser/locales/en-US/chrome/browser/browser.dtd 	1.52
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
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+
Whiteboard: [Branch checkin needed]
Whiteboard: [Branch checkin needed] → [checkin needed (1.8 branch)]
mozilla/browser/locales/en-US/chrome/browser/browser.dtd 	1.25.2.23
Keywords: fixed1.8.1
Whiteboard: [checkin needed (1.8 branch)]
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.
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
(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
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.
(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.
(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.
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 18 is irrelevant here.  Please see comment 17.
You need to log in before you can comment on or make changes to this bug.