The default bug view has changed. See this FAQ.

Make the cache checkbox default-on in the new calendar dialog

RESOLVED FIXED in 4.0.0.1

Status

Calendar
Dialogs
RESOLVED FIXED
5 years ago
2 years ago

People

(Reporter: Vincent (caméléon), Assigned: Fallen)

Tracking

(Depends on: 1 bug)

Trunk
4.0.0.1
Dependency tree / graph

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

5 years ago
Created attachment 636486 [details]
screenshot of the problem (french locale).png

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:13.0) Gecko/20100101 Firefox/13.0.1
Build ID: 20120614114901

Steps to reproduce:

Create a new calendar
Select "On the network" and see the windows


Actual results:

The last line is only a checkbox called "cache". No explanation,  no tooltip. I wonder what I am supposed to do here


Expected results:

Some text should describe what does this option mean in order to help the user
(Assignee)

Comment 1

5 years ago
The Cache really boils down to offline support, so we should just rename the label to that effect. This will also give us more coverage, since users will likely want offline support in any case.

I guess we could go one step further and set this checkbox to on by default. This would bring us one step closer to always using the cache, but still give users the possibility to turn it off.
Summary: Creating remote calendar should indicate what "cache" mean → Rename "Cache" to "Offline Support" and make it default-on in the new event dialog.
Whiteboard: [good first bug][mentor=Fallen][lang=js]
(Reporter)

Comment 2

5 years ago
Hi Philipp,
I completely agree with you.

Comment 3

5 years ago
Enabling cache by default is Bug 462277 that still has some depending bugs open. In addition there might be more bugs that should be evaluated: https://bugzilla.mozilla.org/buglist.cgi?quicksearch=%3Acalendar%20summary%3Acache
(Assignee)

Comment 4

5 years ago
Taking care of the label part in bug 780990. I'm leaving this open for defalult-on the cache in the new calendar dialog.

I'd like to see bug 780992 and bug 700637 fixed before fixing this one though, so removing the good-first-bug part.
Summary: Rename "Cache" to "Offline Support" and make it default-on in the new event dialog. → Make the cache checkbox default-on in the new calendar dialog
Whiteboard: [good first bug][mentor=Fallen][lang=js]

Updated

5 years ago
Depends on: 462277
(Assignee)

Comment 5

2 years ago
Stefan, I think we should attempt this for Lightning 4.0. We want to move towards the cache and may get broader testing this way. What do you think?
Flags: needinfo?(ssitter)
(Assignee)

Comment 6

2 years ago
Created attachment 8564515 [details] [diff] [review]
Fix - v1

To speed things up, I went ahead and created a patch. I'm still open to discussion though.

This patch does go a bit further than the bug title describes and already half-fixes bug 462277: It will remove the cache checkbox from the new calendar wizard and default the cache to on.

To fix bug 462277 we would have to remove the checkbox from the calendar properties dialog too and after that gradually remove all the uncached calendar code.

I think this is a sensible step at the moment, we will get more testing and users can still disable offline support if they think its causing problems.
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Flags: needinfo?(ssitter)
Attachment #8564515 - Flags: review?(ssitter)
(Assignee)

Updated

2 years ago
Target Milestone: --- → 4.0

Comment 7

2 years ago
Comment on attachment 8564515 [details] [diff] [review]
Fix - v1

I did not look at the patch yet. Enabling the offline cache by default might be done to get some more testing to stabilize the feature. But just from reading bugmail it seems that currently users still experience problems when offline cache is enabled. Therefore I think that removing the option to create calendar without offline cache is no good idea. Maybe this can be done once the feature is stable.
Attachment #8564515 - Flags: review?(ssitter) → feedback-
(Assignee)

Comment 8

2 years ago
Ok, fair enough. So would you agree to keeping the checkbox in the dialog, but checking it by default? Aside from that, we should devise a plan to get the feature to be stable. Do you have any more issues in mind then those referenced in bug 462277?
Flags: needinfo?(ssitter)
(Assignee)

Updated

2 years ago
Blocks: 1153752
(Assignee)

Comment 9

2 years ago
Created attachment 8600608 [details] [diff] [review]
Fix - v2

Ok, this patch only does default-on and correctly saves state of the checkbox between provider changes.
Attachment #8564515 - Attachment is obsolete: true
Flags: needinfo?(ssitter)
Attachment #8600608 - Flags: review?(mohit.kanwal)
Attachment #8600608 - Flags: feedback?(ssitter)
Comment on attachment 8600608 [details] [diff] [review]
Fix - v2

Review of attachment 8600608 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me.
Attachment #8600608 - Flags: review?(mohit.kanwal) → review+
(Assignee)

Updated

2 years ago
Keywords: checkin-needed
(Assignee)

Comment 11

2 years ago
Comment on attachment 8600608 [details] [diff] [review]
Fix - v2

Review of attachment 8600608 [details] [diff] [review]:
-----------------------------------------------------------------

We'll need to push this soon, setting checkin-needed. ssitter, if you have comments feel free to bring them in any time!
Attachment #8600608 - Flags: feedback?(ssitter)
Attachment #8600608 - Flags: approval-calendar-beta+
Attachment #8600608 - Flags: approval-calendar-aurora+
(Assignee)

Comment 12

2 years ago
Pushed to comm-central changeset 277d5337de93
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: 4.0 → 4.3
(Assignee)

Comment 13

2 years ago
Backported to releases/comm-aurora changeset 5af28a1ea344
Target Milestone: 4.3 → 4.2
(Assignee)

Comment 14

2 years ago
Backported to releases/comm-beta changeset 36189f77dfe9
Target Milestone: 4.2 → 4.1
(Assignee)

Comment 15

2 years ago
Backported to releases/comm-beta changeset 8d40ecd6bb9f (THUNDERBIRD_38_VERBRANCH)
Target Milestone: 4.1 → 4.0
You need to log in before you can comment on or make changes to this bug.