Homec4science

Determine unread count correctly when rendering notification panel

Authored by epriestley <git@epriestley.com> on Jun 20 2012, 22:20.

Description

Determine unread count correctly when rendering notification panel

Summary:
Currently, when rendering the panel we count the number of unread notifications in the last 15, but this means we can never render a number larger than 15. If the user has more unread notifications than that, or unread notifications older than the most recent 15, there will be a flash of the higher number and then it will update to the lower number afterward.

Instead, count all unread notifications. This uses the same method used to render both numbers.

Test Plan: Loaded a page, checked the menu, nothing exploded.

Reviewers: btrahan, jungejason, vrana

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T974

Differential Revision: https://secure.phabricator.com/D2811

Details

Committed
epriestley <git@epriestley.com>Jun 20 2012, 22:20
Pushed
aubortJan 31 2017, 17:16
Parents
rPH26ca5cbbe357: phd - on start command accurately detect if daemons are running before erroring…
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHaddc2c876471: Determine unread count correctly when rendering notification panel (authored by epriestley <git@epriestley.com>).Jun 20 2012, 22:20