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)

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.