697413 - Usage of progress bar for visualizing imap quota is irritating - details window only

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect)

Description

[Patrick Julian]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:7.0.1) Gecko/20100101 Firefox/7.0.1
Build ID: 20110928134238

Steps to reproduce:

Tested under Mac OS X (10.6.8) with v7.0.1
Open an IMAP email account with given quota of 500MB


Actual results:

in the status bar at the bottom of the main window, a progress bar indicates the amount of the quota that is consumed in the account already. The amount percent value is correct, but the progress bar's permanent animation is irritating  


Expected results:

since the described issue is not actually a running process but a rather static value indicator, this should imho not use a progress bar but another more appropriate UI element

Comment 1

[David :Bienvenu]

by animation, do you mean display? Do you actually see it changing?

Comment 2

[Patrick Julian]

I mean that OS X default behaviour: The blue bar consists of blue vertical stripes that slowly move right to left. The value remains the same. It only makes an impression to the user as if something were being computed or such.

I guess it becomes more apparent what I mean when you click the bar. In the modal window you will see the bar in bigger resolution.

Comment 3

[David :Bienvenu]

The quota bar is either green, yellow, or red, and doesn't animate, afaik. It sounds like you're talking about the progress bar, which is blue, and animates.

Comment 4

[Ben Bucksch (:BenB)]

I don't have a mac, but I think we explicitly disabled the native progress bar look in the themes.

Are you using a different theme (skin)?

Comment 5

[Ben Bucksch (:BenB)]

That change was in bug 385175, FWIW.

Comment 6

[Ben Bucksch (:BenB)]

Patrick, did this work for you in previous releases of TB? If so, that would point to a regression by the above mentioned bug.

Comment 7

[Ben Bucksch (:BenB)]

bienvenu, the commit http://hg.mozilla.org/comm-central/rev/6b7bd9c1ac6c , would it appear in TB 7.0.1 or not?

Patrick, could you please test trunk builds of Thunderbird ("Early Bird")?

Comment 8

[Patrick Julian]

Attachment: This is the bar that I was describing

Comment 9

[David :Bienvenu]

(In reply to Ben Bucksch (:BenB) from comment #7)
> bienvenu, the commit http://hg.mozilla.org/comm-central/rev/6b7bd9c1ac6c ,
> would it appear in TB 7.0.1 or not?
No, loooks like the TFV is TB 9.

Sorry, Patrick, I didn't realize that this stuff had changed.

Comment 10

[Ben Bucksch (:BenB)]

Good. My guess is that this is already fixed in trunk.

Comment 11

[Patrick Julian]

Hi guys, no I am currently running the default theme; currently there are not even any other themes installed. In previous releases, I did not even notice the presence of any such bar. I can test an early bird version if further clarification on this is still required.

Comment 12

[Ludovic Hirlimann [:Usul]]

The one I'm seeing is different on Early bird - earlybird is available at https://www.mozilla.org/en-US/thunderbird/channel/

Comment 13

[Ben Bucksch (:BenB)]

> I can test an early bird version if further clarification on this is still required.

Yes, it is.

Comment 14

[Patrick Julian]

Sweet icon :)

OK it renders differently (better) in the status bar in 9.0a2.

However, the details window still renders it the old way. I'll file another attachment.

Comment 15

[Patrick Julian]

Attachment: details window still with the old bar style

Comment 16

[Ben Bucksch (:BenB)]

Thanks for confirming that the original bug is FIXED.

Details window: OK, that's a point.
I think this is a minor bug now, though.

New Objective: Apply fix for bug 385175 to quota in folder details window.

Comment 17

[Patrick Julian]

ok - agreed - thank you guys!

Comment 18

[Richard Marti (:Paenglab)]

Attachment: Copy the QuotaPanel style to the folder properties

The quota in the folder properties has with this patch the same appearance as the QuotPanel in the statusbar. I only gave the normal color because I haven't found code which gives a warning or critical state.

I had to add a stylesheet definition on folderProps.xul under mailnews. I checked with Seamonkey and saw no problem with this stylesheet definition under it.

Comment 19

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 570465 [details] [diff] [review]
Copy the QuotaPanel style to the folder properties

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

Aside from the question below, I think this looks good (at least on Windows).  r=me, and ui-r=me.

::: mail/themes/gnomestripe/mail/preferences/preferences.css
@@ -94,0 +94,11 @@
> > +
> > +/* Quota meter in Folder Properties */
> > +
> > +#quotaPercentageBar {
NaN more ...

So, why not have the same high/medium/low colours in the folder properties, too?

Comment 20

[Richard Marti (:Paenglab)]

(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #19)
> So, why not have the same high/medium/low colours in the folder properties,
> too?

As I wrote in comment 18: I only gave the normal color because I haven't found code which gives a warning or critical state.

Do you see somewhere code which detects the warning or critical levels? I have only at GMail an IMAP account with quota and with only 2% filled I'm unable to check this.

Comment 21

[David :Bienvenu]

BenB probably remembers better than I do, but I believe it's yellow over a certain level and red after that. There are thresholds specified in prefs - look at mail.quota.* in the config editor.

Also, use mxr to look for UpdateStatusQuota

Comment 22

[Richard Marti (:Paenglab)]

(In reply to David :Bienvenu from comment #21)
> BenB probably remembers better than I do, but I believe it's yellow over a
> certain level and red after that.

I think you have seen this in the status bar where this styling is applied. The progress bar in the folder properties had no special styling, so I don't think it had other colors.

Better we're waiting for BenB for an answer.

Comment 23

[David :Bienvenu]

I believe you're right - it's possible that we'd need to do more work to expose this state to the theme for the folder properties dialog.

Comment 24

[Ben Bucksch (:BenB)]

I don't think there's code to mark it "warning"/"critical" for the folderProperties, only the main window status bar. I personally don't think it's important either, because the bar is much bigger and pretty obvious as-is, while the small status bar needs additional hints.

Comment 25

[Richard Marti (:Paenglab)]

I'm setting checkin-needed because I think bwintons question is answered.

Comment 26

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/806680a1df40