Closed Bug 419351 Opened 12 years ago Closed 12 years ago

Redundant view refresh if tasks are not shown in view

Categories

(Calendar :: Calendar Views, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Fallen, Assigned: Fallen)

Details

Attachments

(1 file)

Attached patch Fix v1Splinter Review
When the unifinder-todo is refreshed (i.e hide completed tasks), the view is refreshed when the value changed. This only needs to be done when tasks are shown in the view.

After review, please also take care of checkin as I will be away until thursday. It would be nice to have this in 0.8.
Attachment #305400 - Flags: review?(Berend.Cornelius)
Flags: wanted-calendar0.8+
Comment on attachment 305400 [details] [diff] [review]
Fix v1

I am not quite happy that we change the hideCompleted State of the calendarviews within the code that belongs to the unifinder-todo, but this your patch certainly improves the situation.
Attachment #305400 - Flags: review?(Berend.Cornelius) → review+
The patch was checked in today. Berend, Philipp, can we close this bug report?
Target Milestone: --- → 0.8
Thanks for checking in,

Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.