Closed
Bug 433619
Opened 16 years ago
Closed 15 years ago
Reminder flag for compact folder popup ineffective when clicking "NO"
Categories
(MailNews Core :: Backend, defect, P3)
MailNews Core
Backend
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b2
People
(Reporter: eddy_nigg, Assigned: mkmelin)
Details
Attachments
(2 files, 2 obsolete files)
7.02 KB,
patch
|
neil
:
review+
neil
:
superreview+
clarkbw
:
ui-review+
|
Details | Diff | Splinter Review |
15.43 KB,
image/png
|
Details |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b5) Gecko/2008032619 Firefox/3.0b5 Build Identifier: 3.0a2pre (2008051203) Occasional popup window questioning if to compact the current folder keeps appearing even after flagging the select box "Do this from now on automatically" and after clicking "NO". Doesn't remember the decision. Reproducible: Always Steps to Reproduce: 1. Set Preferences -> Advanced -> Network & Disk Space -> Disk Space to "ON" and some low value. 2. Wait for the popup appear 3. Select the flag and select "NO" 4. Wait for the next popup appear over and over again Actual Results: Keeps bugging about compacting the folder. Doesn't remember previous decision. Expected Results: Should stop bugging about it. Should remember the decision after selecting the flag to do this from now on automatically. Also the phrase next to the flag could be improved, because it somehow expects that compacting the folder is the only logical decision. Intrusive and annoying :-)
Assignee | ||
Updated•16 years ago
|
Status: UNCONFIRMED → NEW
Component: Preferences → MailNews: Backend
Ever confirmed: true
OS: Linux → All
Product: Thunderbird → Core
QA Contact: preferences → backend
Hardware: PC → All
Target Milestone: --- → mozilla1.9.1a1
Assignee | ||
Comment 1•16 years ago
|
||
Change this to similar wording/functionality to what thunderbird uses for Offline | Work Offline Bad idea to have the "this" in the checkbox wording mean different things depending on what you click.
Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Attachment #329285 -
Flags: superreview?(neil)
Attachment #329285 -
Flags: review?(neil)
Assignee | ||
Comment 2•16 years ago
|
||
Assignee | ||
Updated•16 years ago
|
Priority: -- → P3
Comment 3•16 years ago
|
||
Comment on attachment 329285 [details] [diff] [review] proposed fix > if (!buttonPressed) >- { > okToCompact = PR_TRUE; >- if (checkValue) >- branch->SetBoolPref(PREF_MAIL_PURGE_ASK, PR_FALSE); >- } >+ if (!checkValue) // [ ] Always ask me before compacting folders automatically >+ branch->SetBoolPref(PREF_MAIL_PURGE_ASK, PR_FALSE); I don't like this change. It means that if you don't want to compact folders, you could accidentally prevent it from asking you next time.
Attachment #329285 -
Flags: superreview?(neil)
Attachment #329285 -
Flags: superreview-
Attachment #329285 -
Flags: review?(neil)
Assignee | ||
Comment 4•16 years ago
|
||
Well, only if you bother unchecking the checkbox first, which makes the accident a bit of a stretch. Honoring the checkbox only for "yes" would make this dialog work a different way than the thunderbird going offline ones like http://mxr.mozilla.org/seamonkey/source/mail/base/content/mail-offline.js#243
Updated•16 years ago
|
Product: Core → MailNews Core
Assignee | ||
Comment 5•16 years ago
|
||
... and so does the "Empty Junk"/"Empty Trash" do it. I'm not sure why you're worried about an accidentally disabling the dialog? Honoring the checkbox even with "No" what this bug was about mainly. Frankly I think very few users use this option, and having this dialog pop up all the time would seem to be very annoying. Then again, maybe the wording on the buttons should also be improved to use verbs? Do you wish to compact all local and offline folders to save disk space? [x] Always ask me before compacting folder automatically [ Cancel ] [ Compact now ]
Assignee | ||
Comment 6•16 years ago
|
||
Of course, in the long run we should probably remove the UI (bug 286888) but there are various issues which makes it not possible just yet - see comments in that bug.
Assignee | ||
Comment 7•15 years ago
|
||
Attachment #329285 -
Attachment is obsolete: true
Attachment #329286 -
Attachment is obsolete: true
Attachment #355844 -
Flags: ui-review?(clarkbw)
Attachment #355844 -
Flags: superreview?(neil)
Attachment #355844 -
Flags: review?(neil)
Assignee | ||
Comment 8•15 years ago
|
||
Assignee | ||
Updated•15 years ago
|
Target Milestone: mozilla1.9.1a1 → Thunderbird 3.0b2
Updated•15 years ago
|
Attachment #355844 -
Flags: ui-review?(clarkbw) → ui-review+
Comment 9•15 years ago
|
||
Comment on attachment 355844 [details] [diff] [review] proposed fix, v2 action text buttons! looks much better
Comment 10•15 years ago
|
||
Comment on attachment 355844 [details] [diff] [review] proposed fix, v2 >- if (checkValue) >+ if (!checkValue) // [ ] Always ask me before compacting folders automatically This behaviour change could do with a better variable name, e.g. alwaysAsk, and possibly a better locale string name too, e.g. autoCompactAlwaysAsk (but feel free to think of even better names than my suggestions).
Attachment #355844 -
Flags: superreview?(neil)
Attachment #355844 -
Flags: superreview+
Attachment #355844 -
Flags: review?(neil)
Attachment #355844 -
Flags: review+
Assignee | ||
Comment 11•15 years ago
|
||
changeset: 1614:09e5816364bf http://hg.mozilla.org/comm-central/rev/09e5816364bf ->FIXED
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•