Page MenuHomePhabricator
Files F15422081

D7671.id17323.diff
No OneTemporary
Actions

D7671.id17323.diff
View Options

Index: src/__celerity_resource_map__.php
===================================================================
--- src/__celerity_resource_map__.php
+++ src/__celerity_resource_map__.php
@@ -3419,7 +3419,7 @@
),
'phabricator-notification-menu-css' =>
array(
- 'uri' => '/res/c6b17cfb/rsrc/css/application/base/notification-menu.css',
+ 'uri' => '/res/48b02e64/rsrc/css/application/base/notification-menu.css',
'type' => 'css',
'requires' =>
array(
@@ -4343,7 +4343,7 @@
), array(
'packages' =>
array(
- 'ca6d2b1e' =>
+ '77044f6f' =>
array(
'name' => 'core.pkg.css',
'symbols' =>
@@ -4392,7 +4392,7 @@
41 => 'phabricator-tag-view-css',
42 => 'phui-list-view-css',
),
- 'uri' => '/res/pkg/ca6d2b1e/core.pkg.css',
+ 'uri' => '/res/pkg/77044f6f/core.pkg.css',
'type' => 'css',
),
'2c1dba03' =>
@@ -4584,15 +4584,15 @@
),
'reverse' =>
array(
- 'aphront-dialog-view-css' => 'ca6d2b1e',
- 'aphront-error-view-css' => 'ca6d2b1e',
- 'aphront-list-filter-view-css' => 'ca6d2b1e',
- 'aphront-pager-view-css' => 'ca6d2b1e',
- 'aphront-panel-view-css' => 'ca6d2b1e',
- 'aphront-table-view-css' => 'ca6d2b1e',
- 'aphront-tokenizer-control-css' => 'ca6d2b1e',
- 'aphront-tooltip-css' => 'ca6d2b1e',
- 'aphront-typeahead-control-css' => 'ca6d2b1e',
+ 'aphront-dialog-view-css' => '77044f6f',
+ 'aphront-error-view-css' => '77044f6f',
+ 'aphront-list-filter-view-css' => '77044f6f',
+ 'aphront-pager-view-css' => '77044f6f',
+ 'aphront-panel-view-css' => '77044f6f',
+ 'aphront-table-view-css' => '77044f6f',
+ 'aphront-tokenizer-control-css' => '77044f6f',
+ 'aphront-tooltip-css' => '77044f6f',
+ 'aphront-typeahead-control-css' => '77044f6f',
'differential-changeset-view-css' => '1084b12b',
'differential-core-view-css' => '1084b12b',
'differential-inline-comment-editor' => '5e9e5c4e',
@@ -4606,7 +4606,7 @@
'differential-table-of-contents-css' => '1084b12b',
'diffusion-commit-view-css' => '7aa115b4',
'diffusion-icons-css' => '7aa115b4',
- 'global-drag-and-drop-css' => 'ca6d2b1e',
+ 'global-drag-and-drop-css' => '77044f6f',
'inline-comment-summary-css' => '1084b12b',
'javelin-aphlict' => '2c1dba03',
'javelin-behavior' => '3e3be199',
@@ -4681,56 +4681,56 @@
'javelin-util' => '3e3be199',
'javelin-vector' => '3e3be199',
'javelin-workflow' => '3e3be199',
- 'lightbox-attachment-css' => 'ca6d2b1e',
+ 'lightbox-attachment-css' => '77044f6f',
'maniphest-task-summary-css' => '49898640',
- 'phabricator-action-list-view-css' => 'ca6d2b1e',
- 'phabricator-application-launch-view-css' => 'ca6d2b1e',
+ 'phabricator-action-list-view-css' => '77044f6f',
+ 'phabricator-application-launch-view-css' => '77044f6f',
'phabricator-busy' => '2c1dba03',
'phabricator-content-source-view-css' => '1084b12b',
- 'phabricator-core-css' => 'ca6d2b1e',
- 'phabricator-crumbs-view-css' => 'ca6d2b1e',
+ 'phabricator-core-css' => '77044f6f',
+ 'phabricator-crumbs-view-css' => '77044f6f',
'phabricator-drag-and-drop-file-upload' => '5e9e5c4e',
'phabricator-dropdown-menu' => '2c1dba03',
'phabricator-file-upload' => '2c1dba03',
- 'phabricator-filetree-view-css' => 'ca6d2b1e',
- 'phabricator-flag-css' => 'ca6d2b1e',
+ 'phabricator-filetree-view-css' => '77044f6f',
+ 'phabricator-flag-css' => '77044f6f',
'phabricator-hovercard' => '2c1dba03',
- 'phabricator-jump-nav' => 'ca6d2b1e',
+ 'phabricator-jump-nav' => '77044f6f',
'phabricator-keyboard-shortcut' => '2c1dba03',
'phabricator-keyboard-shortcut-manager' => '2c1dba03',
- 'phabricator-main-menu-view' => 'ca6d2b1e',
+ 'phabricator-main-menu-view' => '77044f6f',
'phabricator-menu-item' => '2c1dba03',
- 'phabricator-nav-view-css' => 'ca6d2b1e',
+ 'phabricator-nav-view-css' => '77044f6f',
'phabricator-notification' => '2c1dba03',
- 'phabricator-notification-css' => 'ca6d2b1e',
- 'phabricator-notification-menu-css' => 'ca6d2b1e',
+ 'phabricator-notification-css' => '77044f6f',
+ 'phabricator-notification-menu-css' => '77044f6f',
'phabricator-object-selector-css' => '1084b12b',
'phabricator-phtize' => '2c1dba03',
'phabricator-prefab' => '2c1dba03',
'phabricator-project-tag-css' => '49898640',
- 'phabricator-remarkup-css' => 'ca6d2b1e',
+ 'phabricator-remarkup-css' => '77044f6f',
'phabricator-shaped-request' => '5e9e5c4e',
- 'phabricator-side-menu-view-css' => 'ca6d2b1e',
- 'phabricator-standard-page-view' => 'ca6d2b1e',
- 'phabricator-tag-view-css' => 'ca6d2b1e',
+ 'phabricator-side-menu-view-css' => '77044f6f',
+ 'phabricator-standard-page-view' => '77044f6f',
+ 'phabricator-tag-view-css' => '77044f6f',
'phabricator-textareautils' => '2c1dba03',
'phabricator-tooltip' => '2c1dba03',
- 'phabricator-transaction-view-css' => 'ca6d2b1e',
- 'phabricator-zindex-css' => 'ca6d2b1e',
- 'phui-button-css' => 'ca6d2b1e',
- 'phui-form-css' => 'ca6d2b1e',
- 'phui-form-view-css' => 'ca6d2b1e',
- 'phui-header-view-css' => 'ca6d2b1e',
- 'phui-icon-view-css' => 'ca6d2b1e',
- 'phui-list-view-css' => 'ca6d2b1e',
- 'phui-object-item-list-view-css' => 'ca6d2b1e',
- 'phui-property-list-view-css' => 'ca6d2b1e',
- 'phui-spacing-css' => 'ca6d2b1e',
- 'sprite-apps-large-css' => 'ca6d2b1e',
- 'sprite-gradient-css' => 'ca6d2b1e',
- 'sprite-icons-css' => 'ca6d2b1e',
- 'sprite-menu-css' => 'ca6d2b1e',
- 'sprite-status-css' => 'ca6d2b1e',
- 'syntax-highlighting-css' => 'ca6d2b1e',
+ 'phabricator-transaction-view-css' => '77044f6f',
+ 'phabricator-zindex-css' => '77044f6f',
+ 'phui-button-css' => '77044f6f',
+ 'phui-form-css' => '77044f6f',
+ 'phui-form-view-css' => '77044f6f',
+ 'phui-header-view-css' => '77044f6f',
+ 'phui-icon-view-css' => '77044f6f',
+ 'phui-list-view-css' => '77044f6f',
+ 'phui-object-item-list-view-css' => '77044f6f',
+ 'phui-property-list-view-css' => '77044f6f',
+ 'phui-spacing-css' => '77044f6f',
+ 'sprite-apps-large-css' => '77044f6f',
+ 'sprite-gradient-css' => '77044f6f',
+ 'sprite-icons-css' => '77044f6f',
+ 'sprite-menu-css' => '77044f6f',
+ 'sprite-status-css' => '77044f6f',
+ 'syntax-highlighting-css' => '77044f6f',
),
));
Index: src/applications/notification/controller/PhabricatorNotificationListController.php
===================================================================
--- src/applications/notification/controller/PhabricatorNotificationListController.php
+++ src/applications/notification/controller/PhabricatorNotificationListController.php
@@ -15,8 +15,8 @@
$nav = new AphrontSideNavFilterView();
$nav->setBaseURI(new PhutilURI('/notification/'));
- $nav->addFilter('all', 'All Notifications');
- $nav->addFilter('unread', 'Unread Notifications');
+ $nav->addFilter('all', pht('All Notifications'));
+ $nav->addFilter('unread', pht('Unread Notifications'));
$filter = $nav->selectFilter($this->filter, 'all');
$pager = new AphrontPagerView();
@@ -50,29 +50,35 @@
$no_data);
}
- $view = phutil_tag_div('phabricator-notification-list', $view);
+ $view = id(new PHUIBoxView())
+ ->addPadding(PHUI::PADDING_MEDIUM)
+ ->addClass('phabricator-notification-list')
+ ->appendChild($view);
- $panel = new AphrontPanelView();
- $panel->setHeader($header);
- $panel->setWidth(AphrontPanelView::WIDTH_FORM);
- $panel->addButton(
- javelin_tag(
+ $subheader = javelin_tag(
'a',
array(
'href' => '/notification/clear/',
- 'class' => 'button',
'sigil' => 'workflow',
),
- 'Mark All Read'));
- $panel->appendChild($view);
- $panel->appendChild($pager);
+ pht('Mark All Read'));
- $nav->appendChild($panel);
+ $notif_header = id(new PHUIHeaderView())
+ ->setHeader($header)
+ ->setSubheader($subheader);
- return $this->buildStandardPageResponse(
+ $box = id(new PHUIObjectBoxView())
+ ->setHeader($notif_header)
+ ->appendChild($view);
+
+ $nav->appendChild($box);
+ $nav->appendChild($pager);
+
+ return $this->buildApplicationPage(
$nav,
array(
- 'title' => 'Notifications',
+ 'title' => pht('Notifications'),
+ 'device' => true,
));
}
Index: webroot/rsrc/css/application/base/notification-menu.css
===================================================================
--- webroot/rsrc/css/application/base/notification-menu.css
+++ webroot/rsrc/css/application/base/notification-menu.css
@@ -38,6 +38,9 @@
.phabricator-notification {
padding: 6px;
+}
+
+.phabricator-notification-menu .phabricator-notification {
cursor: pointer;
}
@@ -53,10 +56,6 @@
color: {$lightgreytext};
}
-.phabricator-notification-list {
- font-size: 11px;
-}
-
.phabricator-notification-list .phabricator-notification-unread,
.phabricator-notification-menu .phabricator-notification-unread {
background: #eceff5;

File Metadata

Mime Type
text/plain
Expires
Sun, Mar 23, 5:20 AM (1 w, 6 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7717934
Default Alt Text
D7671.id17323.diff (8 KB)

Event Timeline

Phabricator · Built by Phacility