Page Menu
Home
Phabricator
Search
Configure Global Search
Log In
Files
F14402480
D15073.id36396.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Award Token
Flag For Later
Size
2 KB
Referenced Files
None
Subscribers
None
D15073.id36396.diff
View Options
diff --git a/resources/celerity/map.php b/resources/celerity/map.php
--- a/resources/celerity/map.php
+++ b/resources/celerity/map.php
@@ -500,7 +500,7 @@
'rsrc/js/core/phtize.js' => 'd254d646',
'rsrc/js/phui/behavior-phui-dropdown-menu.js' => '54733475',
'rsrc/js/phui/behavior-phui-object-box-tabs.js' => '2bfa2836',
- 'rsrc/js/phui/behavior-phui-profile-menu.js' => 'bf2c93d6',
+ 'rsrc/js/phui/behavior-phui-profile-menu.js' => 'bfc2e675',
'rsrc/js/phuix/PHUIXActionListView.js' => 'b5c256b8',
'rsrc/js/phuix/PHUIXActionView.js' => '8cf6d262',
'rsrc/js/phuix/PHUIXAutocomplete.js' => '21dc9144',
@@ -649,7 +649,7 @@
'javelin-behavior-pholio-mock-view' => 'fbe497e7',
'javelin-behavior-phui-dropdown-menu' => '54733475',
'javelin-behavior-phui-object-box-tabs' => '2bfa2836',
- 'javelin-behavior-phui-profile-menu' => 'bf2c93d6',
+ 'javelin-behavior-phui-profile-menu' => 'bfc2e675',
'javelin-behavior-policy-control' => 'ae45872f',
'javelin-behavior-policy-rule-editor' => '5e9f347c',
'javelin-behavior-project-boards' => 'ba4fa35c',
@@ -1774,7 +1774,7 @@
'javelin-util',
'javelin-request',
),
- 'bf2c93d6' => array(
+ 'bfc2e675' => array(
'javelin-behavior',
'javelin-stratcom',
'javelin-dom',
diff --git a/webroot/rsrc/js/phui/behavior-phui-profile-menu.js b/webroot/rsrc/js/phui/behavior-phui-profile-menu.js
--- a/webroot/rsrc/js/phui/behavior-phui-profile-menu.js
+++ b/webroot/rsrc/js/phui/behavior-phui-profile-menu.js
@@ -6,7 +6,18 @@
*/
JX.behavior('phui-profile-menu', function(config) {
- var menu_node = JX.$(config.menuID);
+ // NOTE: This behavior is not initialized in the rendering pipeline for the
+ // menu, so it can get initialized when we build but do not render a menu
+ // (for example, when a page like the panel edit page only has items in
+ // the mobile/application menu, and does not show the profile menu). For now,
+ // just bail if we can't find the menu.
+
+ try {
+ var menu_node = JX.$(config.menuID);
+ } catch (ex) {
+ return;
+ }
+
var collapse_node = JX.$(config.collapseID);
var is_collapsed = config.isCollapsed;
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Tue, Dec 24, 12:39 AM (16 h, 45 m)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6922461
Default Alt Text
D15073.id36396.diff (2 KB)
Attached To
Mode
D15073: Fix "choose icon" on profile menu items
Attached
Detach File
Event Timeline
Log In to Comment