Page MenuHomePhabricator

Connection info in Notifications header needs rethinking
Closed, ResolvedPublic

Description

Previously we had "Mark all Read" in this position, an action I took daily. It looks like that action has moved, but I don't think the connection information here is useful and can be fairly confusing (like now where it says error - what does error mean, do I need to fix something or bug my admin?). T5537 makes it pretty impossible to mark notifications as read with this change.

undefined (124×428 px, 14 KB)

Revisions and Commits

Restricted Differential Revision

Event Timeline

chad claimed this task.
chad raised the priority of this task from to Normal.
chad updated the task description. (Show Details)
chad added a project: Notifications.
chad updated the task description. (Show Details)
chad added subscribers: chad, epriestley, joshuaspence.

(You should be able to hover over the "Connection Error" text to get a more useful description)

But yes, this may not be the best place for this.

chad added a revision: Restricted Differential Revision.Aug 14 2014, 8:29 PM
chad added a commit: Restricted Diffusion Commit.

Closed by commit rPdc69c4e58c2a.