Closed
Bug 1070491
Opened 10 years ago
Closed 9 years ago
Today pane/sidebar: Month and year always "Jan 5555" on startup
Categories
(Calendar :: Calendar Frontend, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
4.0.0.1
People
(Reporter: aryx, Assigned: bv1578)
References
Details
Attachments
(1 file, 2 obsolete files)
2.67 KB,
patch
|
bv1578
:
review+
|
Details | Diff | Splinter Review |
Thunderbird Daily + Lightning Trunk 20140919 on Windows 8.1 Launching Thunderbird with Lightning enabled shows "Jan 5555" as month and year in the Today pane sidebar. Last good: 20140723 First bad: 20140814 No Windows builds between these dates and the Linux one I tried from 20140805 seemed broken.
Reporter | ||
Comment 1•10 years ago
|
||
document.getElementById("monthNameContainer").selectedIndex changes for me while calendar-multiday-view.xml's onResize runs: let size; if (self.orient == "horizontal") { size = scrollbox.boxObject.width; } else { // Here it's 8. size = scrollbox.boxObject.height; // Here it's 0. } Is a Promise responsible for that? Pushlog for comm-central: http://hg.mozilla.org/comm-central/pushloghtml?startdate=2014-07-23&enddate=2014-08-15
Don't know if it's related, but that one is the same point where scrollbox.boxObject.height always returns 0 during startup on SeaMonkey and it causes Bug 1064089.
I tried to look into the issue, but I'm not able to find the real cause. The regression range and the kind of patches applied in the range don't help me so much since the absence of builds for Windows. This patch is only a workaround, it sets the default value for month and year in the first panel of the deck to the date of today (the miniday always shows today on startup) instead of "Jan 5555" so when the bug occurs, the date showed is always correct. If we want a real fix someone else should look into the issue.
Attachment #8532898 -
Flags: review?(philipp)
Comment on attachment 8532898 [details] [diff] [review] Workaround-v1 I've just realized that the same bug occurs on the deck of the weekday (the weekdayNameContainer deck) because on startup its selectedIndex is reset to 0. For this deck it needs something different. Removing the review request.
Attachment #8532898 -
Flags: review?(philipp)
I've also added a workaround for the day label. Overall not an elegant solution, but seeing "Sun Jan 5555" on every startup is by far worse. Philipp, if you know a different way to fix this, please let me know.
Attachment #8532898 -
Attachment is obsolete: true
Attachment #8548481 -
Flags: review?(philipp)
Comment 8•9 years ago
|
||
Comment on attachment 8548481 [details] [diff] [review] Workaround-v2 Review of attachment 8548481 [details] [diff] [review]: ----------------------------------------------------------------- Unfortunately I can't think of a better way to fix this, lets go with what you've done for now. r=philipp
Attachment #8548481 -
Flags: review?(philipp) → review+
Assignee: nobody → bv1578
Attachment #8548481 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8553672 -
Flags: review+
Keywords: checkin-needed
Comment 10•9 years ago
|
||
Pushed to comm-central changeset 1140e1eed436
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 4.0
Comment 11•9 years ago
|
||
Just an update I am still seeing (Jan 5555) this when using TB 37.0b1 and Lightning 3.9b1
Reporter | ||
Comment 12•9 years ago
|
||
As you can see in the 'Target Milestone' field at the top of the bug, this has been fixed in Lightning 4.0, so Lightning 3.9b1 remains affected.
Comment 13•9 years ago
|
||
(In reply to Archaeopteryx [:aryx] from comment #12) > As you can see in the 'Target Milestone' field at the top of the bug, this > has been fixed in Lightning 4.0, so Lightning 3.9b1 remains affected. Thanks I missed that. Will wait for it to be released.
Comment 17•7 years ago
|
||
Seems to have returned in TB 55b2, Lightning 5.7b2. https://support.mozilla.org/en-US/questions/1169131
Comment 18•7 years ago
|
||
(In reply to sfhowes from comment #17) Correct. See Bug 1360914.
You need to log in
before you can comment on or make changes to this bug.
Description
•