Closed
Bug 719034
Opened 13 years ago
Closed 13 years ago
Support CTRL-SHIFT-K as the Lighting Task Quick Filter hotkey.
Categories
(SeaMonkey :: MailNews: General, defect)
SeaMonkey
MailNews: General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philip.chee, Assigned: philip.chee)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
3.34 KB,
patch
|
iannbugzilla
:
review+
mnyromyr
:
superreview+
|
Details | Diff | Splinter Review |
From Bug 316916 Comment 32:
>> So if SeaMonkey implements a dummy "cmd_showQuickFilterBar", the lightning
>> shortcut would work?
> Yes, Seamonkey needs a dummy command with that name and a keyboard shortcut
> that executes that command. The command needs to be exposed via the tabmail's
> command handler.
Assignee | ||
Comment 1•13 years ago
|
||
> This patch works with both the cmd_showQuickFilterBar (Ctrl+Shift+K) and the
> cmd_find (Ctrl+F) commands in Thunderbird. Only the cmd_find command currently
> works in Seamonkey, so that shortcut is used for the placeholder text. If
> Seamonkey implements the QuickFilterBar commands in the future, we would only
> need to update the strings.
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Attachment #589499 -
Flags: review?(iann_bugzilla)
Comment on attachment 589499 [details] [diff] [review]
Patch v1.0 Proposed Hot Key Fix.
> <key key="&focusSearchInput.key;"
> modifiers="accel"
> oncommand="focusElement(document.getElementById('searchInput'));"/>
>
>+ <!-- Needed to support the Lightning Task filter See Bug 316916 -->
>+ <key id="key_qfb_show"
>+ key="&quickFilterBar.show.key2;"
>+ modifiers="accel,shift"
>+ command="cmd_showQuickFilterBar"/>
Nit: indented too far.
Attachment #589499 -
Flags: review?(iann_bugzilla) → review+
Assignee | ||
Comment 3•13 years ago
|
||
> Nit: indented too far.
Fixed locally.
Assignee | ||
Comment 4•13 years ago
|
||
Comment on attachment 589499 [details] [diff] [review]
Patch v1.0 Proposed Hot Key Fix.
For some reason I forgot to ask for moa from Mnyromyr :P
Attachment #589499 -
Flags: superreview?(mnyromyr)
Updated•13 years ago
|
Attachment #589499 -
Flags: superreview?(mnyromyr) → superreview+
Assignee | ||
Comment 5•13 years ago
|
||
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/ad14796cbb77
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•