Use #ifdef in a file to declare shortcut keys in dom/xbl/builtin instead of using ShortcutKeyDefinitions.cpp per platform
Categories
(Core :: DOM: UI Events & Focus Handling, enhancement, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox87 | --- | fixed |
People
(Reporter: masayuki, Assigned: masayuki)
References
Details
Attachments
(15 files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review |
Assignee | ||
Updated•7 years ago
|
Updated•7 years ago
|
Updated•6 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Resetting assignee which I don't work on in this several months.
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
Making the following changes and future changes to the shortcut key definitions
safer, we should have a GTest for checking unexpected changes.
Assignee | ||
Comment 3•4 years ago
|
||
This will have shortcut keys definitions for the other platforms with #if
s.
Depends on D103982
Assignee | ||
Comment 4•4 years ago
|
||
Depends on D103983
Assignee | ||
Comment 5•4 years ago
|
||
Depends on D103984
Assignee | ||
Comment 6•4 years ago
|
||
Depends on D103985
Assignee | ||
Comment 7•4 years ago
|
||
Depends on D103986
Assignee | ||
Comment 8•4 years ago
|
||
Then, we can delete duplicated entries in some platforms.
Depends on D103987
Assignee | ||
Comment 9•4 years ago
|
||
For making the definitions easier to read, same definition across platforms
should be merged. For doing it and making each change review easier, we should
same things for each key group.
Depends on D103988
Assignee | ||
Comment 10•4 years ago
|
||
Depends on D103989
Updated•4 years ago
|
Assignee | ||
Comment 11•4 years ago
|
||
Depends on D103990
Assignee | ||
Comment 12•4 years ago
|
||
Depends on D103991
Assignee | ||
Comment 13•4 years ago
|
||
Depends on D103992
Assignee | ||
Comment 14•4 years ago
|
||
Depends on D103993
Assignee | ||
Comment 15•4 years ago
|
||
Depends on D103994
Assignee | ||
Comment 16•4 years ago
|
||
Depends on D103995
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 17•4 years ago
|
||
Comment 18•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/332db3910c59
https://hg.mozilla.org/mozilla-central/rev/82fb047175b2
https://hg.mozilla.org/mozilla-central/rev/04ca138667bb
https://hg.mozilla.org/mozilla-central/rev/4da1a785ff65
https://hg.mozilla.org/mozilla-central/rev/57910e3df62a
https://hg.mozilla.org/mozilla-central/rev/f85c65d972a3
https://hg.mozilla.org/mozilla-central/rev/8673c1c954e6
https://hg.mozilla.org/mozilla-central/rev/84e422db7674
https://hg.mozilla.org/mozilla-central/rev/0b0a21b506b4
https://hg.mozilla.org/mozilla-central/rev/2c28bee095f9
https://hg.mozilla.org/mozilla-central/rev/80eae543a485
https://hg.mozilla.org/mozilla-central/rev/d05f359fb25c
https://hg.mozilla.org/mozilla-central/rev/c5a9cd3aef19
https://hg.mozilla.org/mozilla-central/rev/c5cd3697e6d8
https://hg.mozilla.org/mozilla-central/rev/7da855a143d6
Description
•