80384 - Keybinding to Remove Text Styles (Accel+Shift+T) does not work on linux

Status: VERIFIED FIXED

(SeaMonkey :: Composer, defect, P3)

Description

[Ninoschka Baca]

Build 2001-05-11-08: Linux RH 6.2

Overview: This is a problem only on linux. The following menu item works but the
keybinding does not:
- Remove Text Styles, Accel+Shift+T

Comment 1

[Ninoschka Baca]

More information on why this is happening only on linux. Copied from bug# 76917:

"Ninoschka Baca--this is very interesting.  I asked around and got this reply:

Yes, Ctrl-Shift-t should be consumed by a ComposeIM module in European
locales, and no application which uses the input method can get it.
This is true on Solaris, too.

See /usr/X11R6/lib/X11/locale/iso8859-1/Compose. This is for iso8859-1 
locales: en_US and wetern european, you will see the entries starting with
"Ctrl<T>". For example,  the first entry:

        Ctrl<T> <plus> <plus> : "#" numbersign

means: when you type Ctrl-Shift-t followed by two plus(+) keys, the
input method should compose "#" as a string and XK_numbersign as a keysym.

Toshi


Ninoschka Baca--I am going to resolve this bug as fixed since the keybindings are 
all working everywhere except ctrl-shift-t on Linux and Solaris (it does work on 
HPUX) due to this IME conflict..."

Comment 2

[Kathleen :Brade]

So Accel-Shift-T is needed for IME on Linux so we should probably:
 * consider looking for a new keybinding in Composer for Remove Text Styles
 * make note of the keybinding for other places which may want to use it

Aaron--can you please add a note about that to your key spec webpage?  thanks!

Comment 3

[Charles Manske]

It's not clear to me from the above description why Ctrl+Shift+T should be
inhibited from working. What exactly is it doing in Linux IME?

Comment 4

[rubydoo123]

moving to 9.2

Comment 5

[Ninoschka Baca]

Mail's 3pane uses Accel+Shift+T to "Get All New Messages" which gets messages 
for all accounts in a profile that are logged in. This also does not work on 
linux.

Comment 6

[Syd Logan]

spam composer change

Comment 7

[Aaron Leventhal]

We need a new key for this apparently, does anyone (Akkana especially) have a
suggestion?

Comment 8

[timeless]

accel-shift-y anyone?

Comment 9

[Charles Manske]

Timeless: not very mnemonic, but then we seem to be out of options!
Won't it be difficult to have a separate keybinding for just Linux, or are we
proposing changing it for all platforms?

Comment 10

[Kathleen :Brade]

accel-shift-Y isn't bad since there is a "y" in the middle of style
We should make this keybinding change consistently across all platforms to ease
platform transition issues (as well as techPubs' job).

Comment 11

[Charles Manske]

I agree. accel+shift+y is easier to reach anyway! This is trivial to fix.
Let's do in now! (Taking bug since I'm mucking in these files more than brade)

Comment 12

[Charles Manske]

Attachment: Fix

Comment 13

[Charles Manske]

Comment on attachment 71507 [details] [diff] [review]
Fix

sr=sfraser

Comment 14

[Asa Dotzler [:asa]]

Comment on attachment 71507 [details] [diff] [review]
Fix

a=asa (on behalf of drivers) for checkin to the 1.0 trunk

Comment 15

[Charles Manske]

checked in

Comment 16

[Dawn Endico]

removed the item for this bug from the release notes for 0.9.9 and beyond,
because the bug is fixed now. Let me know if you think the item should be 
re-added.

Comment 17

[sujay]

verified in 5/22 trunk build on linux. 

ctrl-shift-Y works