Page MenuHomePhabricator
Differential D18485

Update notification UI a little
ClosedPublic
Actions

Authored by chad on Aug 28 2017, 9:49 PM.
Tags
None
Referenced Files
F13198115: D18485.id44404.diff
Mon, May 13, 4:14 AM
F13198075: D18485.id.diff
Mon, May 13, 3:23 AM
F13192747: D18485.id44404.diff
Sun, May 12, 9:34 AM
F13192226: D18485.diff
Sun, May 12, 6:23 AM
F13173939: D18485.diff
Tue, May 7, 9:29 PM
Unknown Object (File)
Mon, May 6, 5:15 AM
Unknown Object (File)
Sat, May 4, 7:08 AM
Unknown Object (File)
Sun, Apr 28, 5:39 AM
View All 89 Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T8944: Unread vs read notifications are difficult to distinguish when viewed at an indirect angle on a 13" Macbook Air
Commits
rPed75250f1a9d: Update notification UI a little
Summary

Fixes T8944. Adds a small dot if notification is new along with color. Goes away when clicked. Increased font and padding for readability.

Test Plan

Send notifications from test account, review them in menu, application search, and in real-time display.

Diff Detail

Repository
rP Phabricator
Branch
notifications (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18191
Build 24460: Run Core Tests
Build 24459: arc lint + arc unit

Revision Contents
Changeset List

PathSize
resources/
celerity/
10 lines
src/
view/
phui/
16 lines
webroot/
rsrc/
css/
aphront/
2 lines
application/
base/
42 lines

Diff 44404

View Options

resources/celerity/map.php

Loading...
View Options

src/view/phui/PHUIFeedStoryView.php

Loading...
View Options

webroot/rsrc/css/aphront/notification.css

Loading...
View Options

webroot/rsrc/css/application/base/notification-menu.css

Loading...
Phabricator · Built by Phacility