composer: change modifier+[] and (smaller/larger) bindings

VERIFIED FIXED in mozilla0.9.1

Status

()

Core
Editor
VERIFIED FIXED
17 years ago
16 years ago

People

(Reporter: Jesse Ruderman, Assigned: Kathleen Brade)

Tracking

Trunk
mozilla0.9.1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
Composer's format menu shows ctrl+shift+[ for smaller and ctrl+shift+] for 
larger, but those shortcuts don't do anything.  The menu items do work.

Comment 1

17 years ago
Over to brade.
Assignee: beppe → brade
(Assignee)

Comment 2

17 years ago
I'm not sure why we want the shift key on this keybinding.  It seems to me that 
the shift key just makes it more difficult for users to use it (not to mention 
anyone who might be physically impaired).

Would anyone mind if we changed this to be ctrl-[ and ctrl-]?

AaronL?  Charley?
Status: NEW → ASSIGNED
OS: Windows 98 → All
Hardware: PC → All
Target Milestone: --- → mozilla0.9.1

Comment 3

17 years ago
[] aren't intl safe iirc, but {} would be even less so.

Can't we be consistent w/ navigator? [ctrl--, ctrl-+ i think]
(Assignee)

Comment 4

17 years ago
Here is my new suggestion:
  Format > Size > Smaller    accel -
  Format > Size > Larger     accel +
  Format > Increase Indent   accel ]
  Format > Decrease Indent   accel [

Comment 5

17 years ago
I totally agree with Kathy's last suggestion.

Comment 6

17 years ago
sounds reasonable to me.
Summary: composer: ctrl+shift+[] (smaller/larger) doesn't work → composer: change modifier+[] and (smaller/larger) bindings

Updated

17 years ago
Blocks: 76927
(Assignee)

Comment 7

17 years ago
Created attachment 33382 [details] [diff] [review]
keybinding changes

Comment 8

17 years ago
r=cmanske
Do you want to use the "FIX IN HAND" notation in status whiteboard?

Comment 9

17 years ago
sr=kin@netscape.com
(Assignee)

Comment 10

17 years ago
fixed
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
(Assignee)

Comment 11

17 years ago
by the way, for verification purposes, it's really:
    accel - 
and accel = (since "+" would require pressing the shift key)

Comment 12

17 years ago
Accel-+ should also work, though -- it doesn't require the shift key if you use
the + on the keypad, and keypad +/- work for changing font sizes in the browser.

Comment 13

17 years ago
verified .
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.