Closed Bug 273600 Opened 20 years ago Closed 16 years ago

Task Progress bar

Categories

(Calendar :: Tasks, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: lucas.charles, Assigned: Fallen)

References

Details

Attachments

(5 files, 1 obsolete file)

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0

We should have have a progress bar as well as progress status for the tasks
it may be very usefull.
Thanks a lot

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Actual Results:  
No progress bar

Expected Results:  
Have a progress bar for the tasks
Where would the progress bar be displayed?
In the tasks table, as background to the "% Complete" column for every task?
Summary: Progress bar → Task Progress bar
The progress bar should be placed in the %complete column as a background
Moreover the user could make the choice of display % complete value to see only
the progress bar.

thanks a lot for replying
*** Bug 256712 has been marked as a duplicate of this bug. ***
QA Contact: gurganbl → sunbird
Reassigning all automatically assigned bugs from Mostafa to nobody@m.o

Bugspam filter: TorontoMostafaMove
Assignee: mostafah → nobody
Component: Sunbird Only → Tasks
OS: Linux → All
QA Contact: sunbird → tasks
Hardware: PC → All
I think this would clutter the UI too much, if you want to have an overview how finished your tasks are, then you can always sort by % completed.

My vote would be for WONTFIX [ extension fodder ]. This could probably be taken care of in an extension.

Charles, no hard feelings please. If you are interested, I can guide you to how to create an extension for this. Please contact me via email in that case.
Status: UNCONFIRMED → RESOLVED
Closed: 16 years ago
Resolution: --- → WONTFIX
Summary: Task Progress bar → Task Progress bar [extension fodder]
(In reply to comment #5)
> I think this would clutter the UI too much, if you want to have an overview how
> finished your tasks are, then you can always sort by % completed.
> 
I would tend to say that it's would simplify the UI. I rarely want to change my sort order to see which task is almost done and which still need lot of work. instead I sort by finish date and look at the %completed to see that info, but you almost have to sort all of this in your head instead of just looking at a progress bar that visually tell in a glimpse which task are which one.

> My vote would be for WONTFIX [ extension fodder ]. This could probably be taken
> care of in an extension.

shouldn't we ask more the community before doing that? considering it was in the proposal with the task mode and from what I remember no one discard it at that time[1]. It might need a little more discussion before choosing a WONTFIX

[1] in bug 253396   see also http://wiki.mozilla.org/Calendar:Task_Mode that was in comment #6
(In reply to comment #6)
> I would tend to say that it's would simplify the UI. I rarely want to change my
> sort order to see which task is almost done and which still need lot of work.
> instead I sort by finish date and look at the %completed to see that info, but
> you almost have to sort all of this in your head instead of just looking at a
> progress bar that visually tell in a glimpse which task are which one.
I'm just arguing, that a progress bar needs much more room that 2-3 digits and a % sign. While this is fine in the task view, its not so good for the small task tree in the today pane or in sunbird.

> shouldn't we ask more the community before doing that? considering it was in
> the proposal with the task mode and from what I remember no one discard it at
> that time[1]. It might need a little more discussion before choosing a WONTFIX
> 
> [1] in bug 253396   see also http://wiki.mozilla.org/Calendar:Task_Mode that
> was in comment #6
We had very many unconfirmed bugs and instead of asking on each bug, I made some decisions on certain bugs. I'm sorry, I have made the wrong decision here. Thanks for pointing that out, you are of course correct then. I didn't have the full task view in mind, just the small one.
Status: RESOLVED → UNCONFIRMED
Resolution: WONTFIX → ---
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: Task Progress bar [extension fodder] → Task Progress bar
Attached patch Add Progressmeters β€” β€” Splinter Review
As a small compensation, here the fix ;-)
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #320694 - Flags: ui-review?(christian.jansen)
Attachment #320694 - Flags: review?(Berend.Cornelius)
(In reply to comment #7)
> I'm just arguing, that a progress bar needs much more room that 2-3 digits and
> a % sign. While this is fine in the task view, its not so good for the small
> task tree in the today pane or in sunbird.
>
agree with that.. in my case I didn't tough about the mini-task-unifinder. having seen your patch screenshot, you added a progress bar to both of them. I know it's the same object so a change to one affect the other, but if the %completed column is too small a progress bar isn't useful and in the mini-task-uni people would like to have a small column with a text %. this might be accomplish by showing text if the column is smaller then 20px and progress bar if longer. 

> We had very many unconfirmed bugs and instead of asking on each bug, I made
> some decisions on certain bugs. I'm sorry, I have made the wrong decision here.

yeah, I saw some of them and your are mostly right and it's something that needs to be done.
Comment on attachment 320694 [details] [diff] [review]
Add Progressmeters

Thanks for the patch. One minor glitch: The progress bar does not reflect the visual style of the system (tested on MAC OSX 10.5) (see attachment). But it would be OK for me, if we fix this in a separate bug. r=christian
Attachment #320694 - Flags: ui-review?(christian.jansen) → ui-review+
Attached image black progress bars β€”
Could you provide a screenshot of what they should look like? I didn't change any progress bar styles at all.
Attached image MAC OSX Progress bar β€”
I'd recommend to check how the progress bar is displayed in Firefox 2 and Firefox 3 and compare it to Sunbird. Maybe the current look is already toolkit-native and will be fixed with the Cocoa widget adaption happened on Trunk.
patch looks good and works almost fine. I have nothing to object.
>+          // The progress mode for a given cell. This method is only called for
>+          // columns of type |progressmeter|.
>+          getProgressMode: function mTV_getProgressMode(aRow, aCol) {
>+              switch(aCol.element.getAttribute("anonid")) {
>+                  case "calendar-task-tree-col-percentcomplete":
>+                      return Components.interfaces.nsITreeView.PROGRESS_NORMAL;
>+              }
>+
>+              return Components.interfaces.nsITreeView.PROGRESS_NONE;
I would use an if- statement instead of "switch" in this case but I am fine with this, too.
On nasty peculiarity I found can be seen in the attached graphic: Under Linux you have a repaint error when you minimize the column. As can be seen the progressmeter stretches in this case over all adjacent columns to the right.
Attached image Screenshot of the repaint error (obsolete) β€”
Comment on attachment 320694 [details] [diff] [review]
Add Progressmeters

As discussed with philipp on irc setting a minwidth at the column resoves the bad repaint behaviour. r=berend
Attachment #320694 - Flags: review?(Berend.Cornelius) → review+
Added the minwidth.

Unfortunately, I can't really influence the progressmeter in the tree much. I could change some colors, but that would be about it. The only user of the progressmeter in a <tree> is the xpfe download managers, so I believe not many people will be interested in fixing it.

Christian, if you provide me some colors (and border colors), I can add a fix for this.

Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 16 years ago16 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.9
Btw, I used a switch statement, in case we want to take care of other fields in the future. This makes the transition easier then.
Attachment #321049 - Attachment is obsolete: true
Christian, I set a minwidth on the cell. You are still getting the repaint error with the checked in patch? We might have to increase the minwidth.
No, I don't get the repaint error. Sorry for the confusion....
Cheked in lightning 2008061918 and Sunbird 20080619 -> VERIFIED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: