All day events don't have the start/end time widget disabled
Categories
(Calendar :: Dialogs, defect)
Tracking
(thunderbird_esr91 wontfix)
Tracking | Status | |
---|---|---|
thunderbird_esr91 | --- | wontfix |
People
(Reporter: rachel, Assigned: rachel)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
darktrojan
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Firefox/91.0
Steps to reproduce:
Create an event, make it all day. As soon as you click the "all day" check box, the start and end time input should be disabled like it was in TB 78. The same happens when editing an all day event.
Assignee | ||
Comment 1•3 years ago
|
||
Alice, could you please find the regression. This is very straight forward. Working in TB 78, not working in TB 91, of 95 beta. Didn't try Daily.
Comment 2•3 years ago
|
||
I can reproduce using 95.0b5 if you mean not dimmed, indicating it is disabled.
I can't make any changes to the times, only the date, and would consider that as disabled.
Assignee | ||
Comment 3•3 years ago
|
||
Yes, the time picker appears, but changes are discarded. Please compare to the TB 78 behaviour.
Comment 4•3 years ago
|
||
(In reply to Rachel Martin from comment #3)
Yes, the time picker appears, but changes are discarded. Please compare to the TB 78 behaviour.
Discarded means the same as disabled to me. I can't make a time change, never paid attention to how it was in previous versions because I was creating an All Day event.
Assignee | ||
Comment 5•3 years ago
|
||
Likely broken here:
https://searchfox.org/comm-central/diff/9dae3c5143be50c5102e0ddf937db2e128eee5c2/calendar/lightning/content/lightning-item-iframe.js#1961
https://searchfox.org/comm-central/diff/e3cb9833ac6db1db6d0d52d634b0c1274d4d8463/calendar/lightning/content/lightning-item-iframe.js#1975
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 6•3 years ago
|
||
Comment 7•3 years ago
|
||
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/b4d4250e3f1d
Fix non-disabled time widgets for all day events, use set/removeAttribute() instead of toggleAttribute(). r=darktrojan
Updated•3 years ago
|
Description
•