401521 - Day, Week and Multiweek View changes to Month View after restart

Status: VERIFIED FIXED

(Calendar :: Lightning Only, defect)

Description

[Pete Riley]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.7) Gecko/20070914 Firefox/2.0.0.7
Build Identifier: Lightning 0.8pre 2007-10-28-05 / Thunderbird 2.0.0.6

 

Reproducible: Always

Steps to Reproduce:
1. In Calendar Mode, show multiweek view.
2. Restart Thunderbird.
3. Go to Calendar Mode again.
Actual Results:  
I see month view.

Expected Results:  
I should see multiweek view.

Comment 1

[Omar Bajraszewski]

Confirmed. I can see it also for day and week view 

Comment 2

[Omar Bajraszewski]

Works with 2007102703
Fails with 2007102808

Comment 3

[Omar Bajraszewski]

Same bug 386480 was fixed for Lt 0.7 pre

Comment 4

[Stefan Sitter [:ssitter]]

Probably regression from Bug 390508.

Comment 5

[Philipp Kewisch [:Fallen][☁️📆]]

Attachment: Persist Calendar View - v1

This patch takes care of persisting the calendar view (again), unified for sunbird and lightning.

Comment 6

[Michael Büttner (no reviews TFN)]

Comment on attachment 286555 [details] [diff] [review]
Persist Calendar View - v1

>+        var viewNode = deck.childNodes[deck.selectedIndex];
>+        var viewName = viewNode.id.replace(/-view/, "");
>+        showCalendarView(viewName);
>+    }
Probably consolidate this into something like:
          showCalendarView(
            deck.childNodes[deck.selectedIndex]
              .id.replace(/-view/, ""));

Indeed a nice unified solution, r=mickey.

Comment 7

[Philipp Kewisch [:Fallen][☁️📆]]

I don't really like formatting the line like this, I think the way I put it is more readable.

Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED

Comment 8

[Omar Bajraszewski]

verified with Lt 2007103103