Page MenuHomePhabricator
Differential D7671

Mobile Notifications
ClosedPublic
Actions

Authored by chad on Nov 28 2013, 11:13 PM.
Tags
None
Referenced Files
F13257455: D7671.id17374.diff
Sun, May 26, 1:21 AM
F13241286: D7671.id17373.diff
Wed, May 22, 8:00 PM
F13224694: D7671.id.diff
Sun, May 19, 10:10 AM
F13214957: D7671.id17373.diff
Fri, May 17, 2:00 PM
Unknown Object (File)
May 1 2024, 10:06 PM
Unknown Object (File)
Apr 16 2024, 11:50 PM
Unknown Object (File)
Apr 10 2024, 4:23 AM
Unknown Object (File)
Feb 12 2024, 5:05 PM
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