Closed Bug 1428849 Opened 3 years ago Closed 3 years ago

Remove datetimepicker binding and related ones

Categories

(Toolkit :: XUL Widgets, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: ntim, Assigned: ntim)

References

Details

(Whiteboard: [xbl-unused])

Attachments

(1 file)

I don't see it being used anywhere. So assume only thunderbird/seamonkey are using it.

https://dxr.mozilla.org/mozilla-central/search?q=datetimepicker&redirect=true

Most of the instances refer to the content date time picker, which is a XUL panel with an iframe to an HTML time picker (which does not use the XBL datetimepicker binding).
Assignee: nobody → ntim.bugs
Notifying :Paenglab for potential comm-central changes.
Flags: needinfo?(richard.marti)
Whiteboard: [xbl-unused]
Lightning uses his own bindings but TB and SM use them in the Address book for the birthday setting. So we need to port them to c-c.

Thanks for the info.
Flags: needinfo?(richard.marti)
Comment on attachment 8940806 [details]
Bug 1428849 - Remove datetimepicker.xml bindings.

https://reviewboard.mozilla.org/r/211070/#review217160

Thanks! Seems like a sane removal. I'm also pleased that this will help remove some of the confusion between this binding and the DOM datepicker code.

I can't find usage within mozilla-central outside of the tests you removed. I know that there is some usage still in comm-central, but I suspect those folks will be able to move this over to c-c without much trouble.

Assuming a green try push, r=me. Thanks!

::: commit-message-72100:1
(Diff revision 3)
> +Bug 1428849 - Remove datetimepicker binding and related ones. r=mconley

Nit: "related ones" is kinda vague in this context. Perhaps "Remove datetimepicker binding and descendant bindings". Even "Remove datetimepicker binding" is probably sufficient.
Attachment #8940806 - Flags: review?(mconley) → review+
Pushed by ntim.bugs@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/3eba56d0b50f
Remove datetimepicker.xml bindings. r=mconley
Depends on: 1429231
Depends on: 1429232
https://hg.mozilla.org/mozilla-central/rev/3eba56d0b50f
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Type: enhancement → task
You need to log in before you can comment on or make changes to this bug.