Closed
Bug 448289
Opened 16 years ago
Closed 16 years ago
Cannot tag message with digits on shifted top row of AZERTY keyboard (keypad digits work OK)
Categories
(MailNews Core :: Internationalization, defect)
MailNews Core
Internationalization
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b2
People
(Reporter: paulgirard, Assigned: philor)
References
(Blocks 1 open bug)
Details
(Keywords: intl, regression, Whiteboard: [key hell][no l10n impact])
Attachments
(1 file)
(deleted),
patch
|
mkmelin
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; fr; rv:1.9.0.1) Gecko/2008070206 Firefox/3.0.1
Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; fr-FR; rv:1.9.1a2pre) Gecko/2008072800 Lightning/0.6a1 Shredder/3.0a2pre
when one try to use the tag hotkey for instance "Remove All Tags" by pressing the azerty key combination SHIFT+à (=0), the tag is not apllied.
If one use the "verr num" capability of the keyboard, by pressing the key "," the tag is applied correctly.
Reproducible: Always
Steps to Reproduce:
1. open Thunderbird (Shredder v3.0a2pre)
2. select any mail
3. on an azerty keyboard, press SHIFT+& (=1)
4. The message is not tagged as IMPORTANT
Actual Results:
The message is not tagged as IMPORTANT
Expected Results:
The message should be tagged in regards to the tag hotkeys configuration
0 : remove all tag
1 : IMPORTANT
etc...
I have lightning and gcal provider add-ons but I don't think there's anything linked with this.
Comment 1•16 years ago
|
||
does this happen with TB v2?
No, add a flag in TB v2 works fine with AZERTY keyboard.
Paul
Comment 4•16 years ago
|
||
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b1pre) Gecko/20081003 SeaMonkey/2.0a2pre
using Belgian AZERTY keyboard (where digits on top row are shifted)
Hit 1 (i.e. Shift+&) => no effect
Hit Keypad-1 (in NumLock mode) => Message line in List pane becomes red
I'm not sure what the comma refers to in relation with "verr num" in comment #0 -- note that "verrouillage numérique" or "verr num" is French for Num Lock.
Moving to MailNews Core (because common to Tb and Sm) but not sure I found the right "Component".
Component: Mail Window Front End → Database
OS: Mac OS X → All
Product: Thunderbird → MailNews Core
QA Contact: front-end → database
Hardware: Macintosh → All
Comment 5•16 years ago
|
||
P.S. Build ID: 20081003001423
Updated•16 years ago
|
Summary: azerty keyboard hotkeys 0,1,2,3,4,5 using SHIFT don't work to tag message wherease "verr num" is fine → Cannot tag message with numbers on shifted top row of AZERTY keyboard (keypad numbers work OK)
Updated•16 years ago
|
Summary: Cannot tag message with numbers on shifted top row of AZERTY keyboard (keypad numbers work OK) → Cannot tag message with digits on shifted top row of AZERTY keyboard (keypad digits work OK)
Comment 6•16 years ago
|
||
This sounds very much like bug 359638 (the Core regression) -> bug 429219 (the FF fix) > bug 432002 (the "wanted" SM port)...
Assignee: nobody → smontagu
Status: UNCONFIRMED → NEW
Component: Database → Internationalization
Ever confirmed: true
QA Contact: database → mailnews.i18n
Updated•16 years ago
|
Blocks: tb-tagsmeta
Updated•16 years ago
|
Flags: blocking-thunderbird3?
Comment 8•16 years ago
|
||
Simon, are you planning on working on this bug?
Flags: blocking-thunderbird3? → blocking-thunderbird3+
Target Milestone: --- → Thunderbird 3.0b3
Comment 9•16 years ago
|
||
If this is the same as bug 359638/bug 429219, Masayuki is probably a better owner than me.
Assignee: smontagu → masayuki
Comment 10•16 years ago
|
||
Oh, this needs to change core widget code on all platforms. Because we don't send alternative keycodes by gecko keypress event without modifier key pressing...
So, the changes may be too risky for Fx3.1. Are we using different branch for Tb3 and Fx3.1?
Whiteboard: [key hell]
Comment 11•16 years ago
|
||
(In reply to comment #10)
> So, the changes may be too risky for Fx3.1. Are we using different branch for
> Tb3 and Fx3.1?
They will be the same branch (mozilla-1.9.1).
Comment 12•16 years ago
|
||
(In reply to comment #10)
> this needs to change core widget code on all platforms. Because we don't
> send alternative keycodes by gecko keypress event without modifier key
> pressing...
Can/should this be solved by adding modifiers="shift any" to the appropriate key element?
https://developer.mozilla.org/En/XUL/Key
Maybe these are the elements?
http://mxr.mozilla.org/comm-central/source/mailnews/base/resources/content/mailWindowOverlay.xul#405
Or/and these?
http://mxr.mozilla.org/comm-central/source/mail/base/content/mailWindowOverlay.xul#436
Comment 13•16 years ago
|
||
Ah, it's good idea.
Assignee | ||
Comment 14•16 years ago
|
||
Has a faint whiff of hackiness to it, but it's not like using unmodified numbers as commandkeys is one of the platform's most common things.
Assignee: masayuki → philringnalda
Status: NEW → ASSIGNED
Attachment #362509 -
Flags: superreview?(neil)
Attachment #362509 -
Flags: review?(mkmelin+mozilla)
Assignee | ||
Comment 15•16 years ago
|
||
Despite the simplicity of karlt's solution, my memories of key hell the first make me really want to slip this in before b2 for maximum exposure.
Whiteboard: [key hell] → [key hell][no l10n impact]
Target Milestone: Thunderbird 3.0b3 → Thunderbird 3.0b2
Comment 16•16 years ago
|
||
Comment on attachment 362509 [details] [diff] [review]
Fix v.1
Conveniently we don't expose these shortcuts in the UI, instead we leave the user to guess them by making them the access keys in the tags submenu.
Attachment #362509 -
Flags: superreview?(neil) → superreview+
Assignee | ||
Comment 17•16 years ago
|
||
Rub it in, Mr. My Platform Actually Shows Accesskeys In Menus - on OS X, we just mash random combinations while trying to see what might have changed.
Updated•16 years ago
|
Attachment #362509 -
Flags: review?(mkmelin+mozilla) → review+
Comment 18•16 years ago
|
||
Comment on attachment 362509 [details] [diff] [review]
Fix v.1
Looks reasonable, r=mkmelin
Assignee | ||
Comment 19•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•