-
Notifications
You must be signed in to change notification settings - Fork 76
Conversation
By analyzing the blame information on this pull request, we identified @ChristophWurst, @Gomez and @jancborchardt to be potential reviewers |
@@ -10,7 +10,7 @@ | |||
<a class="folder {{#if specialRole}} icon-{{specialRole}} svg{{/if}}"> | |||
{{name}} | |||
{{#if unseen}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We go in only if unseen. Drafts are always seen. So you need to do further tests here.
Thanks for this PR! Please add a changelog enty, too. |
@tahaalibra thanks for that PR. I'd prefer to have that view logic inside the folder view. That can be achieved easily by using template helpers like we did here. Using a view helper is a valid approach, but it makes the code harder to maintain as you'd have the logic split into two separate locations. There's only one template that would make use of it. So simply move the filter code over as template helper and fix the bug @Gomez mentioned, thanks! |
855befb
to
56e0b3d
Compare
show total count for drafts
Updated the code...Using template helpers seems much better option |
Thanks a lot, that looks very good! We'll have to decrement that 'total' number when the user deletes a message. Otherwise the counter stays unchanged while deleting drafts. There's code that updates the 'unseen' value, so we just have to adjust that to also update the other value. |
This is where clicks on the trash icon are handled. You can use |
i think this also need to be updated for another case i.e if a user receives a new message... |
Also the title bar still shows drafts with unseen....i will try to correct these problem...i will contact if i need any help |
[x] the count decreases if we delete a message in the drafts folder one thing though, i checked this without this PR, the unseen count for the index folder does not if suppose the user has loaded the mail page, then he receives a new mail, then he clicks 'check messages'. the unseen message count does not get updated. |
Thanks @tahaalibra! Works now. When i create further drafts, the counter does not reflect this immediately. A complete reload is needed. I will create a issue for it. The "unseen count for the inbox folder" is a known issue. There is a hacky (closed) PR #1047. Will create a further issue (as we dont have one). |
@ChristophWurst And thanks for the explanation! Greatly appreciated! |
this solves issue #540
Currently the drafts shows unseen/unread count. this solution would enable to show total count for the drafts folder only.
review @Gomez @ChristophWurst, This is my first PR, i am open for code review