Tickets :: [#11370] Unread messages count, display options

6.0.0-beta1

9/24/25

Summary	Unread messages count, display options
Queue	IMP
Queue Version	6.0.0beta2
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	Thomas (at) nurzen (dot) de
Created	08/21/2012 (4782 days ago)
Due
Updated	08/29/2012 (4774 days ago)
Assigned
Resolved	08/22/2012 (4781 days ago)
Milestone
Patch	No

08/29/2012 12:48:46 PM	Git Commit	Comment #9	Reply to this comment
Changes have been made in Git (master): commit 37b9bedd134e15449457deb31f034dde65d4ef68 Author: Michael M Slusarz <slusarz@horde.org> Date: Wed Aug 22 14:00:39 2012 -0600 Traditional tree should mirror dynamic tree - only show unseen count (~~Request #11370~~) imp/lib/Imap/Tree.php \| 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/37b9bedd134e15449457deb31f034dde65d4ef68

08/23/2012 02:52:29 AM	Michael Slusarz	Comment #8	Reply to this comment
No. $after is not used in generation of the sidebar (in IMP 6) so we can't use it here. Sorry, what? I'm talking about not showing a "(0)" if there are no unseen mails in a folder. The committed code doesn't do this - it only adds the unseen count if $poll_info->unseen is non-zero.

08/22/2012 09:03:48 PM	Thomas (at) nurzen (dot) de	Comment #7	Reply to this comment
No. $after is not used in generation of the sidebar (in IMP 6) so we can't use it here. Sorry, what? I'm talking about not showing a "(0)" if there are no unseen mails in a folder.

08/22/2012 08:51:03 PM	Michael Slusarz	Comment #6	Reply to this comment
Me again, didn't you miss a number 2? No. $after is not used in generation of the sidebar (in IMP 6) so we can't use it here.

08/22/2012 08:42:47 PM	Thomas (at) nurzen (dot) de	Comment #5	Reply to this comment
Me again, didn't you miss a number 2? diff: if ($poll_info->unseen) { $this->unseen += $poll_info->unseen; $label = '<strong>' . $label . '</strong>'; - } + //} - $after = ' (' . $poll_info->unseen . '/' . $poll_info->msgs . ')'; + $after = ' (' . $poll_info->unseen . ')'; + }

08/22/2012 08:31:42 PM	Thomas (at) nurzen (dot) de	Comment #4	Reply to this comment
Thats okay, i hacked it into my installation. Will be updating to "6.0.0final" anyway ;) Thanks for your fast answer. Bye

08/22/2012 08:16:09 PM	Michael Slusarz	Comment #3 Version ⇒ 6.0.0beta2 State ⇒ Resolved Summary ⇒ Unread messages count, display options Assigned to Michael Slusarz	Reply to this comment
Makes sense - we should be consistent with the display between the views. I've implemented this in IMP 6 only, though, since it is a significant UI change and we are only really doing bug fixes in IMP 5.x right now.

08/22/2012 08:14:19 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git (develop): commit 37b9bedd134e15449457deb31f034dde65d4ef68 Author: Michael M Slusarz <slusarz@horde.org> Date: Wed Aug 22 14:00:39 2012 -0600 Traditional tree should mirror dynamic tree - only show unseen count (~~Request #11370~~) imp/lib/Imap/Tree.php \| 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/37b9bedd134e15449457deb31f034dde65d4ef68

08/21/2012 11:04:37 PM	Thomas (at) nurzen (dot) de	Comment #1 Milestone ⇒ State ⇒ New Patch ⇒ No Queue ⇒ IMP Summary ⇒ Unreed messages count, display options Type ⇒ Enhancement Priority ⇒ 1. Low	Reply to this comment
Hello, in the horde menu we have the webmail folders in a tree-style and behind each of them the unreed and total mails are printed out: "MyFolder (11/42)" The enhancement i want to propose would be to introduce a option to modify this in a way so we can: 1. only display the unreed-count and thereby: 2. only show counter(s) when unreed messages in folder exist (like in imp H3 i think) Independant to your thoughts about this proposal, i would be happy to learn where this behaviour could be "hacked" into the system. Greetings an Thanks!