Closed Bug 1428849 Opened 6 years ago Closed 6 years ago

Remove datetimepicker binding and related ones


(Toolkit :: UI Widgets, task)

Not set



Tracking Status
firefox59 --- fixed


(Reporter: ntim, Assigned: ntim)



(Whiteboard: [xbl-unused])


(1 file)

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

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.

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
Remove datetimepicker.xml bindings. r=mconley
Depends on: 1429231
Depends on: 1429232
Closed: 6 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.