Closed Bug 1471850 Opened 3 years ago Closed 3 years ago

Number picker does not work on custom repeat

Categories

(Calendar :: Dialogs, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jorgk-bmo, Assigned: Paenglab)

References

Details

Attachments

(1 file)

Seen on TB 60 beta 9.

STR:
Create event, repeat: Custom, every: xx days. The arrows don't work.
Also below: Create xxx appointments.
Not sure whether the assignee automatically gets e-mail.
Flags: needinfo?(richard.marti)
I thought calendar uses its own bindings for all its time widgets. I checked for all textbox[type="number"] in calendar and added our bindings.css except for the ones in preferences which get this binding already.

I have only added the TB binding because SM has no such binding for textbox[type="number"] and will be probably broken.
Flags: needinfo?(richard.marti)
Attachment #8988555 - Flags: review?(philipp)
Attachment #8988555 - Flags: approval-calendar-esr?(philipp)
Attachment #8988555 - Flags: approval-calendar-beta?(philipp)
Grrrr, spoke too early. On the beta one needs to first remove the installed version, then reset pref extensions.installedDistroAddon.{e2fda1a4-762b-4020-b5ad-a41df1933103}, and upon restart one gets the new version which *IS* working. Sorry about the noise.
Comment on attachment 8988555 [details] [diff] [review]
Bug1471850-textbox-binding.patch

r+a=philipp via IRC. Thanks!
Attachment #8988555 - Flags: review?(philipp) → review+
Attachment #8988555 - Flags: approval-calendar-esr?(philipp)
Attachment #8988555 - Flags: approval-calendar-esr+
Attachment #8988555 - Flags: approval-calendar-beta?(philipp)
Attachment #8988555 - Flags: approval-calendar-beta+
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/80d117100cd2
Use our own binding for textbox[type="number"] in Calendar. r=philipp
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 6.5
You need to log in before you can comment on or make changes to this bug.