Page Menu
Home
Phabricator
Search
Configure Global Search
Log In
Files
F15430315
D17206.id41370.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Award Token
Flag For Later
Size
15 KB
Referenced Files
None
Subscribers
None
D17206.id41370.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
@@ -9,7 +9,7 @@
'names' => array(
'conpherence.pkg.css' => '0b64e988',
'conpherence.pkg.js' => '6249a1cf',
- 'core.pkg.css' => '9c725fa0',
+ 'core.pkg.css' => '1afa1d13',
'core.pkg.js' => 'a2ead3fe',
'darkconsole.pkg.js' => 'e7393ebb',
'differential.pkg.css' => '9535a7e6',
@@ -162,7 +162,7 @@
'rsrc/css/phui/phui-object-box.css' => '6b487c57',
'rsrc/css/phui/phui-pager.css' => 'bea33d23',
'rsrc/css/phui/phui-pinboard-view.css' => '2495140e',
- 'rsrc/css/phui/phui-profile-menu.css' => '4768721a',
+ 'rsrc/css/phui/phui-profile-menu.css' => 'c71ecdcd',
'rsrc/css/phui/phui-property-list-view.css' => '6d8e58ac',
'rsrc/css/phui/phui-remarkup-preview.css' => '1a8f2591',
'rsrc/css/phui/phui-segment-bar-view.css' => '46342871',
@@ -172,7 +172,7 @@
'rsrc/css/phui/phui-timeline-view.css' => 'bc523970',
'rsrc/css/phui/phui-two-column-view.css' => '7babf5b9',
'rsrc/css/phui/workboards/phui-workboard-color.css' => 'b60ef38a',
- 'rsrc/css/phui/workboards/phui-workboard.css' => '16441d5e',
+ 'rsrc/css/phui/workboards/phui-workboard.css' => 'c88912ee',
'rsrc/css/phui/workboards/phui-workcard.css' => '00979e40',
'rsrc/css/phui/workboards/phui-workpanel.css' => 'a3a63478',
'rsrc/css/sprite-login.css' => '587d92d7',
@@ -535,7 +535,6 @@
'rsrc/js/core/phtize.js' => 'd254d646',
'rsrc/js/phui/behavior-phui-dropdown-menu.js' => '1aa4c968',
'rsrc/js/phui/behavior-phui-file-upload.js' => 'b003d4fb',
- 'rsrc/js/phui/behavior-phui-profile-menu.js' => '12884df9',
'rsrc/js/phui/behavior-phui-submenu.js' => 'a6f7a73b',
'rsrc/js/phui/behavior-phui-tab-group.js' => '0a0b10e9',
'rsrc/js/phuix/PHUIXActionListView.js' => 'b5c256b8',
@@ -690,7 +689,6 @@
'javelin-behavior-phui-dropdown-menu' => '1aa4c968',
'javelin-behavior-phui-file-upload' => 'b003d4fb',
'javelin-behavior-phui-hovercards' => 'bcaccd64',
- 'javelin-behavior-phui-profile-menu' => '12884df9',
'javelin-behavior-phui-submenu' => 'a6f7a73b',
'javelin-behavior-phui-tab-group' => '0a0b10e9',
'javelin-behavior-policy-control' => 'd0c516d5',
@@ -883,7 +881,7 @@
'phui-oi-simple-ui-css' => 'a8beebea',
'phui-pager-css' => 'bea33d23',
'phui-pinboard-view-css' => '2495140e',
- 'phui-profile-menu-css' => '4768721a',
+ 'phui-profile-menu-css' => 'c71ecdcd',
'phui-property-list-view-css' => '6d8e58ac',
'phui-remarkup-preview-css' => '1a8f2591',
'phui-segment-bar-view-css' => '46342871',
@@ -894,7 +892,7 @@
'phui-timeline-view-css' => 'bc523970',
'phui-two-column-view-css' => '7babf5b9',
'phui-workboard-color-css' => 'b60ef38a',
- 'phui-workboard-view-css' => '16441d5e',
+ 'phui-workboard-view-css' => 'c88912ee',
'phui-workcard-view-css' => '00979e40',
'phui-workpanel-view-css' => 'a3a63478',
'phuix-action-list-view' => 'b5c256b8',
@@ -1030,11 +1028,6 @@
'javelin-dom',
'javelin-typeahead-normalizer',
),
- '12884df9' => array(
- 'javelin-behavior',
- 'javelin-stratcom',
- 'javelin-dom',
- ),
'13c739ea' => array(
'javelin-behavior',
'javelin-stratcom',
diff --git a/src/__phutil_library_map__.php b/src/__phutil_library_map__.php
--- a/src/__phutil_library_map__.php
+++ b/src/__phutil_library_map__.php
@@ -3422,7 +3422,6 @@
'PhabricatorPolicyTestObject' => 'applications/policy/__tests__/PhabricatorPolicyTestObject.php',
'PhabricatorPolicyType' => 'applications/policy/constants/PhabricatorPolicyType.php',
'PhabricatorPonderApplication' => 'applications/ponder/application/PhabricatorPonderApplication.php',
- 'PhabricatorProfileMenuCollapsedSetting' => 'applications/settings/setting/PhabricatorProfileMenuCollapsedSetting.php',
'PhabricatorProfileMenuEditEngine' => 'applications/search/editor/PhabricatorProfileMenuEditEngine.php',
'PhabricatorProfileMenuEditor' => 'applications/search/editor/PhabricatorProfileMenuEditor.php',
'PhabricatorProfileMenuEngine' => 'applications/search/engine/PhabricatorProfileMenuEngine.php',
@@ -8575,7 +8574,6 @@
),
'PhabricatorPolicyType' => 'PhabricatorPolicyConstants',
'PhabricatorPonderApplication' => 'PhabricatorApplication',
- 'PhabricatorProfileMenuCollapsedSetting' => 'PhabricatorInternalSetting',
'PhabricatorProfileMenuEditEngine' => 'PhabricatorEditEngine',
'PhabricatorProfileMenuEditor' => 'PhabricatorApplicationTransactionEditor',
'PhabricatorProfileMenuEngine' => 'Phobject',
diff --git a/src/applications/search/engine/PhabricatorProfileMenuEngine.php b/src/applications/search/engine/PhabricatorProfileMenuEngine.php
--- a/src/applications/search/engine/PhabricatorProfileMenuEngine.php
+++ b/src/applications/search/engine/PhabricatorProfileMenuEngine.php
@@ -239,11 +239,6 @@
}
}
- $more_items = $this->newAutomaticMenuItems($nav);
- foreach ($more_items as $item) {
- $nav->addMenuItem($item);
- }
-
$nav->selectFilter(null);
$this->navigation = $nav;
@@ -410,73 +405,6 @@
}
}
- private function newAutomaticMenuItems(AphrontSideNavFilterView $nav) {
- $items = array();
-
- // NOTE: We're adding a spacer item for the fixed footer, so that if the
- // menu taller than the page content you can still scroll down the page far
- // enough to access the last item without the content being obscured by the
- // fixed items.
- $items[] = id(new PHUIListItemView())
- ->setHideInApplicationMenu(true)
- ->addClass('phui-profile-menu-spacer');
-
- $collapse_id = celerity_generate_unique_node_id();
- $viewer = $this->getViewer();
- $collapse_key = PhabricatorProfileMenuCollapsedSetting::SETTINGKEY;
-
- $is_collapsed = $viewer->getUserSetting($collapse_key);
-
- if ($is_collapsed) {
- $nav->addClass('phui-profile-menu-collapsed');
- } else {
- $nav->addClass('phui-profile-menu-expanded');
- }
-
- if ($viewer->isLoggedIn()) {
- $settings_uri = '/settings/adjust/?key='.$collapse_key;
- } else {
- $settings_uri = null;
- }
-
- Javelin::initBehavior(
- 'phui-profile-menu',
- array(
- 'menuID' => $nav->getMainID(),
- 'collapseID' => $collapse_id,
- 'isCollapsed' => (bool)$is_collapsed,
- 'settingsURI' => $settings_uri,
- ));
-
- $collapse_icon = id(new PHUIIconCircleView())
- ->addClass('phui-list-item-icon')
- ->addClass('phui-profile-menu-visible-when-expanded')
- ->setIcon('fa-chevron-left');
-
- $expand_icon = id(new PHUIIconCircleView())
- ->addClass('phui-list-item-icon')
- ->addClass('phui-profile-menu-visible-when-collapsed')
- ->addSigil('has-tooltip')
- ->setMetadata(
- array(
- 'tip' => pht('Expand'),
- 'align' => 'E',
- ))
- ->setIcon('fa-chevron-right');
-
- $items[] = id(new PHUIListItemView())
- ->setName('Collapse')
- ->addIcon($collapse_icon)
- ->addIcon($expand_icon)
- ->setID($collapse_id)
- ->addClass('phui-profile-menu-footer')
- ->addClass('phui-profile-menu-footer-1')
- ->setHideInApplicationMenu(true)
- ->setHref('#');
-
- return $items;
- }
-
public function getConfigureURI() {
return $this->getItemURI('configure/');
}
diff --git a/src/applications/settings/setting/PhabricatorProfileMenuCollapsedSetting.php b/src/applications/settings/setting/PhabricatorProfileMenuCollapsedSetting.php
deleted file mode 100644
--- a/src/applications/settings/setting/PhabricatorProfileMenuCollapsedSetting.php
+++ /dev/null
@@ -1,12 +0,0 @@
-<?php
-
-final class PhabricatorProfileMenuCollapsedSetting
- extends PhabricatorInternalSetting {
-
- const SETTINGKEY = 'profile-menu.collapsed';
-
- public function getSettingName() {
- return pht('Profile Menu Collapsed');
- }
-
-}
diff --git a/webroot/rsrc/css/phui/phui-profile-menu.css b/webroot/rsrc/css/phui/phui-profile-menu.css
--- a/webroot/rsrc/css/phui/phui-profile-menu.css
+++ b/webroot/rsrc/css/phui/phui-profile-menu.css
@@ -2,7 +2,7 @@
* @provides phui-profile-menu-css
*/
-.device-desktop .phui-profile-menu-collapsed .phabricator-nav-local {
+.device-desktop .phui-profile-menu .phabricator-nav-local {
width: {$menu.profile.width.collapsed};
max-width: {$menu.profile.width.collapsed};
}
@@ -58,8 +58,7 @@
background-size: 100%;
}
-.phui-profile-menu .phui-profile-menu-collapsed .phabricator-side-menu
- .phui-list-item-href {
+.phui-profile-menu .phabricator-side-menu .phui-list-item-href {
text-align: center;
padding: 42px 4px 14px;
line-height: 14px;
@@ -67,19 +66,17 @@
font-size: {$smallerfontsize};
}
-.phui-profile-menu .phui-profile-menu-collapsed .phabricator-side-menu
- .phui-list-item-name {
+.phui-profile-menu .phabricator-side-menu .phui-list-item-name {
display: block;
overflow: hidden;
text-overflow: ellipsis;
}
-.phui-profile-menu .phui-profile-menu-collapsed .phabricator-side-menu
- .phui-list-item-icon,
-.phui-profile-menu .phui-profile-menu-collapsed .phabricator-side-menu
- .phui-list-item-href .phui-list-item-icon {
- top: 14px;
- left: 32px;
+.phui-profile-menu .phabricator-side-menu .phui-list-item-icon,
+.phui-profile-menu .phabricator-side-menu .phui-list-item-href
+ .phui-list-item-icon {
+ top: 14px;
+ left: 32px;
}
.phui-profile-menu .phabricator-side-menu
@@ -131,14 +128,12 @@
padding: 16px;
}
-.phui-profile-menu .phui-profile-menu-collapsed .phabricator-side-menu
- .phui-profile-menu-error {
+.phui-profile-menu .phabricator-side-menu .phui-profile-menu-error {
padding: 16px 8px;
overflow: hidden;
text-overflow: ellipsis;
}
-
.phui-profile-menu .phabricator-side-menu .phui-list-item-disabled
.phui-list-item-href,
.phui-profile-menu .phui-list-sidenav .phui-list-item-disabled
@@ -153,150 +148,15 @@
padding: 8px 12px 16px;
}
-.phui-profile-menu .phui-profile-menu-collapsed .phabricator-side-menu
- .phui-profile-segment-bar {
+.phui-profile-menu .phabricator-side-menu .phui-profile-segment-bar {
padding: 8px 8px 16px;
}
-
.phui-profile-menu .phabricator-side-menu .phui-profile-menu-spacer {
box-sizing: border-box;
height: {$menu.profile.item.height};
}
-.phui-profile-menu .phui-profile-menu-collapsed .phabricator-side-menu
- .phui-profile-menu-footer
- .phui-list-item-name {
- display: none;
-}
-
-.phui-profile-menu .phui-profile-menu-expanded
- .phui-profile-menu-visible-when-collapsed,
-.phui-profile-menu .phui-profile-menu-collapsed
- .phui-profile-menu-visible-when-expanded {
- display: none;
-}
-
-
-.phui-profile-menu .phui-profile-menu-collapsed .phabricator-side-menu
- .phui-profile-menu-footer .phui-list-item-href {
- padding: 20px 0 24px;
-}
-
-.phui-profile-menu .phabricator-side-menu .phui-profile-menu-footer
- .phui-list-item-href:hover {
- background: transparent;
-}
-
-.phui-profile-menu .phui-profile-menu-collapsed .phabricator-side-menu
- .phui-profile-menu-footer .phui-list-item-href .phui-list-item-icon {
- top: 8px;
-}
-
-.phui-profile-menu .phui-profile-menu-footer .phui-icon-circle {
- border-color: {$darkbluetext};
-}
-
-.phui-profile-menu .phui-profile-menu-footer .phui-icon-circle .phui-icon-view {
- color: {$darkbluetext};
-}
-
-.phui-profile-menu .phui-profile-menu-footer
- .phui-icon-circle.phui-list-item-icon {
- font-size: 12px;
-}
-
-.phui-profile-menu .phabricator-side-menu .phui-profile-menu-footer-1 {
- bottom: 0;
- position: fixed;
-}
-
-.phui-profile-menu .phui-profile-menu-footer-1 {
- width: {$menu.profile.width};
-}
-
-.phui-profile-menu .phui-profile-menu-collapsed .phui-profile-menu-footer-1 {
- width: {$menu.profile.width.collapsed};
-}
-
-.phui-profile-menu .phabricator-side-menu
- .phui-list-item-selected.phui-profile-menu-footer .phui-list-item-href {
- background: transparent;
-}
-
-.phui-profile-menu .phui-profile-menu-collapsing
- .phabricator-nav-local {
- animation: profile-menu-collapse 0.2s;
-}
-
-.phui-profile-menu .phui-profile-menu-expanding
- .phabricator-nav-local {
- animation: profile-menu-expand 0.2s;
-}
-
-.phui-profile-menu .phui-profile-menu-collapsing
- .phabricator-side-menu .phui-list-item-href {
- animation: profile-menu-blink 0.2s;
-}
-
-.phui-profile-menu .phui-profile-menu-expanding
- .phabricator-side-menu .phui-list-item-href {
- animation: profile-menu-blink 0.2s;
-}
-
-@keyframes profile-menu-blink {
- 0% {
- opacity: 1.0;
- }
- 25% {
- opacity: 0.0;
- }
- 75% {
- opacity: 0.0;
- }
- 100% {
- opacity: 1.0;
- }
-}
-
-@keyframes profile-menu-collapse {
- 0% {
- width: {$menu.profile.width};
- max-width: {$menu.profile.width};
- }
- 33% {
- width: {$menu.profile.width};
- max-width: {$menu.profile.width};
- }
- 66% {
- width: {$menu.profile.width.collapsed};
- max-width: {$menu.profile.width.collapsed};
- }
- 100% {
- width: {$menu.profile.width.collapsed};
- max-width: {$menu.profile.width.collapsed};
- }
-}
-
-@keyframes profile-menu-expand {
- 0% {
- width: {$menu.profile.width.collapsed};
- max-width: {$menu.profile.width.collapsed};
- }
- 33% {
- width: {$menu.profile.width.collapsed};
- max-width: {$menu.profile.width.collapsed};
- }
- 66% {
- width: {$menu.profile.width};
- max-width: {$menu.profile.width};
- }
- 100% {
- width: {$menu.profile.width};
- max-width: {$menu.profile.width};
- }
-}
-
!print .phui-profile-menu .phabricator-side-menu {
display: none;
}
diff --git a/webroot/rsrc/css/phui/workboards/phui-workboard.css b/webroot/rsrc/css/phui/workboards/phui-workboard.css
--- a/webroot/rsrc/css/phui/workboards/phui-workboard.css
+++ b/webroot/rsrc/css/phui/workboards/phui-workboard.css
@@ -32,11 +32,6 @@
}
.device-desktop .project-board-wrapper .phui-workboard-view-shadow {
- left: {$menu.profile.width};
-}
-
-.device-desktop .phui-profile-menu-collapsed .project-board-wrapper
- .phui-workboard-view-shadow {
left: {$menu.profile.width.collapsed};
}
diff --git a/webroot/rsrc/js/phui/behavior-phui-profile-menu.js b/webroot/rsrc/js/phui/behavior-phui-profile-menu.js
deleted file mode 100644
--- a/webroot/rsrc/js/phui/behavior-phui-profile-menu.js
+++ /dev/null
@@ -1,53 +0,0 @@
-/**
- * @provides javelin-behavior-phui-profile-menu
- * @requires javelin-behavior
- * javelin-stratcom
- * javelin-dom
- */
-
-JX.behavior('phui-profile-menu', function(config) {
- // 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;
-
- JX.DOM.listen(collapse_node, 'click', null, function(e) {
- is_collapsed = !is_collapsed;
-
- JX.DOM.alterClass(menu_node, 'phui-profile-menu-collapsing', is_collapsed);
- JX.DOM.alterClass(menu_node, 'phui-profile-menu-expanding', !is_collapsed);
-
- var duration = 0.2;
-
- setTimeout(function() {
- JX.DOM.alterClass(menu_node, 'phui-profile-menu-collapsed', is_collapsed);
- JX.DOM.alterClass(menu_node, 'phui-profile-menu-expanded', !is_collapsed);
- }, (duration / 2) * 1000);
-
- setTimeout(function() {
- JX.DOM.alterClass(menu_node, 'phui-profile-menu-collapsing', false);
- JX.DOM.alterClass(menu_node, 'phui-profile-menu-expanding', false);
- }, duration * 1000);
-
-
- if (config.settingsURI) {
- new JX.Request(config.settingsURI)
- .setData({value: (is_collapsed ? 1 : 0)})
- .send();
- }
-
- e.kill();
- });
-
-});
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Tue, Mar 25, 7:01 AM (5 d, 15 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7402352
Default Alt Text
D17206.id41370.diff (15 KB)
Attached To
Mode
D17206: Remove collapsable option from ProfileMenu
Attached
Detach File
Event Timeline
Log In to Comment