Page MenuHomePhabricator

D10966.id26335.diff
No OneTemporary

D10966.id26335.diff

diff --git a/resources/celerity/map.php b/resources/celerity/map.php
--- a/resources/celerity/map.php
+++ b/resources/celerity/map.php
@@ -8,7 +8,7 @@
return array(
'names' => array(
'core.pkg.css' => 'a157f664',
- 'core.pkg.js' => '7c53868c',
+ 'core.pkg.js' => 'e4556c9b',
'darkconsole.pkg.js' => 'df001cab',
'differential.pkg.css' => '8af45893',
'differential.pkg.js' => '42c10e78',
@@ -348,7 +348,7 @@
'rsrc/image/texture/table_header_hover.png' => '038ec3b9',
'rsrc/image/texture/table_header_tall.png' => 'd56b434f',
'rsrc/js/application/aphlict/Aphlict.js' => '4a07e8e3',
- 'rsrc/js/application/aphlict/behavior-aphlict-dropdown.js' => '7d4cc76c',
+ 'rsrc/js/application/aphlict/behavior-aphlict-dropdown.js' => '72118e8b',
'rsrc/js/application/aphlict/behavior-aphlict-listen.js' => 'a826c925',
'rsrc/js/application/aphlict/behavior-aphlict-status.js' => '58f7803f',
'rsrc/js/application/auth/behavior-persona-login.js' => '9414ff18',
@@ -542,7 +542,7 @@
'inline-comment-summary-css' => '8cfd34e8',
'javelin-aphlict' => '4a07e8e3',
'javelin-behavior' => '61cbc29a',
- 'javelin-behavior-aphlict-dropdown' => '7d4cc76c',
+ 'javelin-behavior-aphlict-dropdown' => '72118e8b',
'javelin-behavior-aphlict-listen' => 'a826c925',
'javelin-behavior-aphlict-status' => '58f7803f',
'javelin-behavior-aphront-basic-tokenizer' => 'b3a4b884',
@@ -1278,6 +1278,16 @@
'phabricator-phtize',
'changeset-view-manager',
),
+ '72118e8b' => array(
+ 'javelin-behavior',
+ 'javelin-request',
+ 'javelin-stratcom',
+ 'javelin-vector',
+ 'javelin-dom',
+ 'javelin-uri',
+ 'javelin-behavior-device',
+ 'phabricator-title',
+ ),
'724b1247' => array(
'javelin-behavior',
'javelin-typeahead-ondemand-source',
@@ -1343,16 +1353,6 @@
'javelin-request',
'javelin-router',
),
- '7d4cc76c' => array(
- 'javelin-behavior',
- 'javelin-request',
- 'javelin-stratcom',
- 'javelin-vector',
- 'javelin-dom',
- 'javelin-uri',
- 'javelin-behavior-device',
- 'phabricator-title',
- ),
'7e41274a' => array(
'javelin-install',
),
diff --git a/src/view/page/menu/PhabricatorMainMenuView.php b/src/view/page/menu/PhabricatorMainMenuView.php
--- a/src/view/page/menu/PhabricatorMainMenuView.php
+++ b/src/view/page/menu/PhabricatorMainMenuView.php
@@ -316,11 +316,12 @@
),
$message_count_number);
- $message_icon_tag = phutil_tag(
+ $message_icon_tag = javelin_tag(
'span',
array(
'class' => 'phabricator-main-menu-message-icon phui-icon-view '.
'phui-font-fa fa-comments',
+ 'sigil' => 'menu-icon',
),
'');
@@ -401,11 +402,12 @@
),
$count_number);
- $icon_tag = phutil_tag(
+ $icon_tag = javelin_tag(
'span',
array(
'class' => 'phabricator-main-menu-alert-icon phui-icon-view '.
'phui-font-fa fa-bell',
+ 'sigil' => 'menu-icon',
),
'');
diff --git a/webroot/rsrc/js/application/aphlict/behavior-aphlict-dropdown.js b/webroot/rsrc/js/application/aphlict/behavior-aphlict-dropdown.js
--- a/webroot/rsrc/js/application/aphlict/behavior-aphlict-dropdown.js
+++ b/webroot/rsrc/js/application/aphlict/behavior-aphlict-dropdown.js
@@ -16,6 +16,7 @@
var dropdown = JX.$(config.dropdownID);
var bubble = JX.$(config.bubbleID);
+ var icon = JX.DOM.scry(bubble, 'span', 'menu-icon')[0];
var count;
if (config.countID) {
@@ -69,6 +70,9 @@
if (!e.getNode('phabricator-notification-menu')) {
// Click outside the dropdown; hide it.
JX.DOM.hide(dropdown);
+ if (icon) {
+ JX.DOM.alterClass(icon, 'white', false);
+ }
statics.visible = null;
return;
}
@@ -140,6 +144,10 @@
p.setPos(dropdown);
statics.visible = dropdown;
+
+ if (icon) {
+ JX.DOM.alterClass(icon, 'white', true);
+ }
}
);

File Metadata

Mime Type
text/plain
Expires
Fri, Dec 20, 1:23 PM (3 h, 30 m)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6910489
Default Alt Text
D10966.id26335.diff (4 KB)

Event Timeline