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)

Lightning 3.6
x86_64
Windows 8.1
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
4.0.0.1

People

(Reporter: aryx, Assigned: bv1578)

References

Details

Attachments

(1 file, 2 obsolete files)

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.
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.
Attached patch Workaround-v1 (obsolete) β€” β€” Splinter Review
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)
Version: Trunk → Lightning 3.6
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)
Attached patch Workaround-v2 (obsolete) β€” β€” Splinter Review
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 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+
Attached patch Workaround-v2 for checkin β€” β€” Splinter Review
Assignee: nobody → bv1578
Attachment #8548481 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8553672 - Flags: review+
Keywords: checkin-needed
Pushed to comm-central changeset 1140e1eed436
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 4.0
Just an update
I am still seeing (Jan 5555) this when using TB 37.0b1 and Lightning 3.9b1
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.
(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.
Seems to have returned in TB 55b2, Lightning 5.7b2.

https://support.mozilla.org/en-US/questions/1169131
(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.

Attachment

General

Creator:
Created:
Updated:
Size: