Page MenuHomePhabricator
Differential D7671

Mobile Notifications
ClosedPublic
Actions

Authored by chad on Nov 28 2013, 11:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 23, 4:36 AM
Unknown Object (File)
Mon, Nov 18, 11:32 PM
Unknown Object (File)
Fri, Nov 15, 9:36 AM
Unknown Object (File)
Tue, Nov 12, 7:07 AM
Unknown Object (File)
Mon, Nov 11, 11:44 AM
Unknown Object (File)
Mon, Nov 11, 12:21 AM
Unknown Object (File)
Thu, Nov 7, 11:47 AM
Unknown Object (File)
Fri, Nov 1, 1:25 AM
View All Files
Subscribers
aran
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rPb2debb14c702: Mobile Notifications
Summary

Touch up /notifications/ for desktop and mobile

Test Plan

Tested read and unread notifications on mobile and desktop

Diff Detail

Branch
notif-css
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

chad added a comment.Nov 28 2013, 11:14 PM

{F85899}

{F85900}

epriestley accepted this revision.Dec 1 2013, 2:58 AM

That "Mark All Read" link is really hard for me to identify as an action.

chad added a comment.Dec 1 2013, 3:05 AM

I have some new stuff coming for header actions

chad updated this revision to Unknown Object (????).Dec 3 2013, 8:04 PM
  • Update with new button
chad closed this revision.Dec 3 2013, 8:06 PM

Closed by commit rPb2debb14c702 (authored by @chad).

Revision Contents
Changeset List

PathSize
src/
92 lines
applications/
notification/
controller/
36 lines
webroot/
rsrc/
css/
application/
base/
7 lines

Diff 17323

View Options

src/__celerity_resource_map__.php

Loading...
View Options

src/applications/notification/controller/PhabricatorNotificationListController.php

Loading...
View Options

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

Loading...
Phabricator · Built by Phacility