Remove support for disableKeyNavigation on richlistbox, autocomplete and tree bindings

RESOLVED FIXED in Firefox 66

Status

()

enhancement
P5
normal
RESOLVED FIXED
6 months ago
5 months ago

People

(Reporter: ntim, Assigned: bgrins)

Tracking

(Blocks 2 bugs)

unspecified
mozilla66
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox66 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

6 months ago
This disableKeyNavigation attribute is unused in mozilla-central and is only used in comm-central in the suite/ folder (SeaMonkey).

https://searchfox.org/mozilla-central/search?q=disableKeyNavigation&case=false&regexp=false&path=

Updated

6 months ago
Priority: -- → P5
Assignee

Updated

5 months ago
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED

Comment 2

5 months ago
Pushed by ntim.bugs@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/adad39927668
Remove support for disableKeyNavigation on richlistbox, autocomplete and tree bindings;r=timdream

Comment 3

5 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/adad39927668
Status: ASSIGNED → RESOLVED
Last Resolved: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
(In reply to :ntim (low availability until February) from comment #0)
> is only used in comm-central in the suite/ folder (SeaMonkey).

No it's used in mail and mailnews (=> Thunderbird) too. Use it e.g. for single key actions (move forward, archive etc.) while the tree still have focus.
https://searchfox.org/comm-central/search?q=disableKeyNavigation&path=mail
Reporter

Comment 5

5 months ago
(In reply to Magnus Melin [:mkmelin] from comment #4)
> (In reply to :ntim (low availability until February) from comment #0)
> > is only used in comm-central in the suite/ folder (SeaMonkey).
> 
> No it's used in mail and mailnews (=> Thunderbird) too. Use it e.g. for
> single key actions (move forward, archive etc.) while the tree still have
> focus.
> https://searchfox.org/comm-central/search?q=disableKeyNavigation&path=mail

Oops, I guess it would be possible to reintroduce support for the attribute in the tree binding. Brian, what do you think ?
Flags: needinfo?(bgrinstead)
Reporter

Comment 6

5 months ago
(Since it seems only used in the tree binding in c-c)
That would be great. Since as AIUI, you're going to completely remove <tree> eventually, removing m-c unused properties from it now seems a bit unnecessary to me.

Comment 8

5 months ago
(In reply to Magnus Melin [:mkmelin] from comment #4)

> No it's used in mail and mailnews (=> Thunderbird) too. Use it e.g. for
> single key actions (move forward, archive etc.) while the tree still have
> focus.

Indeed: -> Bug 1515759
Assignee

Comment 9

5 months ago
Sure, it could be reintroduced into <tree> in Bug 1515759
Flags: needinfo?(bgrinstead)
You need to log in before you can comment on or make changes to this bug.