Closed
Bug 953643
Opened 10 years ago
Closed 10 years ago
Groups should have a "+/-" and a count of "online/total"
Categories
(Instantbird Graveyard :: Contacts window, enhancement)
Instantbird Graveyard
Contacts window
Tracking
(Not tracked)
RESOLVED
FIXED
0.2b1
People
(Reporter: romain, Assigned: romain)
Details
Attachments
(1 file, 3 obsolete files)
6.54 KB,
patch
|
Details | Diff | Splinter Review |
*** Original post on bio 196 at 2009-07-01 21:42:00 UTC *** As written in the Summary, groups should have a heading + or - (whether the group is collapsed or not). It could also be interesting to add a counter of (number/total) after the group name. Easy to fix I guess, but can become hard if there is no way to know the exact total.
Comment 1•10 years ago
|
||
*** Original post on bio 196 at 2009-07-01 21:59:19 UTC *** (In reply to comment #0) > It could also be interesting to add a counter of (number/total) after the group > name. Displaying the number of hidden rows when a group is collapsed seems interesting. I don't see the point of displaying the count of visible rows, if they are visible, adding the count of them is useless. The total count is not something interesting if there's no way to display offline buddies, so I guess we shouldn't bother with it until we add a way to show offline buddies.
Assignee | ||
Comment 2•10 years ago
|
||
*** Original post on bio 196 as attmnt 139 at 2009-07-26 23:24:00 UTC *** In this patch, I created a new method "_updateGroupLabel()" which updates the group label, adding the number of hidden buddies at the end of the name. This number is refreshed when a buddy log out or a new buddy log in, and is displayed at start up if the group was closed in the preferences.
Attachment #8351883 -
Flags: review?(florian)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → romain
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•10 years ago
|
||
*** Original post on bio 196 as attmnt 140 at 2009-07-26 23:32:00 UTC *** Sorry... There was a trailing 'z' in the previous patch :(
Attachment #8351884 -
Flags: review?(florian)
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8351883 [details] [diff] [review] proposed patch, without keypress handler. *** Original change on bio 196 attmnt 139 at 2009-07-26 23:32:25 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8351883 -
Attachment is obsolete: true
Attachment #8351883 -
Flags: review?(florian)
Assignee | ||
Comment 5•10 years ago
|
||
*** Original post on bio 196 as attmnt 141 at 2009-07-27 00:28:00 UTC *** A few changes (as seen with Florian on chat). A bug has been seen while doing this patch: - The animation is performed even if the group is closed on adding a new buddy (buddy.xml:146), we must check that the buddy is not collapsed before doing it.
Attachment #8351885 -
Flags: review?(florian)
Assignee | ||
Comment 6•10 years ago
|
||
Comment on attachment 8351884 [details] [diff] [review] patch v2 *** Original change on bio 196 attmnt 140 at 2009-07-27 00:28:10 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8351884 -
Attachment is obsolete: true
Attachment #8351884 -
Flags: review?(florian)
Comment 7•10 years ago
|
||
Comment on attachment 8351885 [details] [diff] [review] patch which follows florian's advice *** Original change on bio 196 attmnt 141 at 2009-07-27 14:28:01 UTC *** Looks good, thanks! Nit: >+ let GroupIsHidden = this.hasAttribute("closed"); >+ if (GroupIsHidden) > buddyElt.setAttribute("collapsed", "true"); Use groupIsHidden, not GroupIsHidden for the variable name.
Attachment #8351885 -
Flags: review?(florian) → review+
Comment 8•10 years ago
|
||
*** Original post on bio 196 as attmnt 142 at 2009-07-27 14:30:00 UTC *** Unfortunately, -moz-appearance: treetwisty looks bad on Mac (excessive margins) and doesn't work at all on Windows. The attached patch is the version I will push.
Comment 9•10 years ago
|
||
Comment on attachment 8351885 [details] [diff] [review] patch which follows florian's advice *** Original change on bio 196 attmnt 141 at 2009-07-27 14:30:23 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8351885 -
Attachment is obsolete: true
Comment 10•10 years ago
|
||
*** Original post on bio 196 at 2009-07-27 23:37:05 UTC *** https://hg.instantbird.org/instantbird/rev/782ea6d56eef Fixed, thanks! (In reply to comment #4) > A bug has been seen while doing this patch: > - The animation is performed even if the group is closed on adding a new > buddy (buddy.xml:146), we must check that the buddy is not collapsed before > doing it. This was filed as bug 953648 (bio 202).
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.2b1
You need to log in
before you can comment on or make changes to this bug.
Description
•