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

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect)

Description

[Jesse Ruderman]

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

[kinmoz]

Over to brade.

Comment 2

[Kathleen :Brade]

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?

Comment 3

[timeless]

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

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

Comment 4

[Kathleen :Brade]

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

Comment 5

[Charles Manske]

I totally agree with Kathy's last suggestion.

Comment 6

[timeless]

sounds reasonable to me.

Comment 7

[Kathleen :Brade]

Attachment: keybinding changes

Comment 8

[Charles Manske]

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

Comment 9

[kinmoz]

sr=[e-mail]

Comment 10

[Kathleen :Brade]

fixed

Comment 11

[Kathleen :Brade]

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

Comment 12

[Akkana Peck]

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

[sujay]

verified .