Page MenuHomePhabricator
Files F14047815

D18274.id43949.diff
No OneTemporary
Actions

D18274.id43949.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
@@ -13,7 +13,7 @@
'core.pkg.js' => '5d80e0db',
'darkconsole.pkg.js' => '1f9a31bc',
'differential.pkg.css' => '45951e9e',
- 'differential.pkg.js' => '414ada25',
+ 'differential.pkg.js' => 'a8dbe99b',
'diffusion.pkg.css' => 'a2d17c7d',
'diffusion.pkg.js' => '6134c5a1',
'favicon.ico' => '30672e08',
@@ -398,7 +398,7 @@
'rsrc/js/application/dashboard/behavior-dashboard-query-panel-select.js' => '453c5375',
'rsrc/js/application/dashboard/behavior-dashboard-tab-panel.js' => 'd4eecc63',
'rsrc/js/application/diff/DiffChangeset.js' => '99abf4cd',
- 'rsrc/js/application/diff/DiffChangesetList.js' => 'cb1570cb',
+ 'rsrc/js/application/diff/DiffChangesetList.js' => 'deec798d',
'rsrc/js/application/diff/DiffInline.js' => 'e83d28f3',
'rsrc/js/application/diff/behavior-preview-link.js' => '051c7832',
'rsrc/js/application/differential/behavior-comment-preview.js' => '51c5ad07',
@@ -778,7 +778,7 @@
'phabricator-darkmessage' => 'c48cccdd',
'phabricator-dashboard-css' => 'fe5b1869',
'phabricator-diff-changeset' => '99abf4cd',
- 'phabricator-diff-changeset-list' => 'cb1570cb',
+ 'phabricator-diff-changeset-list' => 'deec798d',
'phabricator-diff-inline' => 'e83d28f3',
'phabricator-drag-and-drop-file-upload' => '58dea2fa',
'phabricator-draggable-list' => 'bea6e7f4',
@@ -1950,10 +1950,6 @@
'cae95e89' => array(
'syntax-default-css',
),
- 'cb1570cb' => array(
- 'javelin-install',
- 'phuix-button-view',
- ),
'ccf1cbf8' => array(
'javelin-install',
'javelin-dom',
@@ -2035,6 +2031,10 @@
'javelin-typeahead-ondemand-source',
'javelin-dom',
),
+ 'deec798d' => array(
+ 'javelin-install',
+ 'phuix-button-view',
+ ),
'e1d25dfb' => array(
'javelin-behavior',
'javelin-stratcom',
diff --git a/src/applications/differential/view/DifferentialChangesetListView.php b/src/applications/differential/view/DifferentialChangesetListView.php
--- a/src/applications/differential/view/DifferentialChangesetListView.php
+++ b/src/applications/differential/view/DifferentialChangesetListView.php
@@ -298,6 +298,9 @@
'Show All Inlines' => pht('Show All Inlines'),
'List Inline Comments' => pht('List Inline Comments'),
+
+ 'Hide or show all inline comments.' =>
+ pht('Hide or show all inline comments.'),
),
));
diff --git a/webroot/rsrc/js/application/diff/DiffChangesetList.js b/webroot/rsrc/js/application/diff/DiffChangesetList.js
--- a/webroot/rsrc/js/application/diff/DiffChangesetList.js
+++ b/webroot/rsrc/js/application/diff/DiffChangesetList.js
@@ -190,6 +190,10 @@
label = pht('Collapse or expand inline comment.');
this._installKey('q', label, this._onkeycollapse);
+
+ label = pht('Hide or show all inline comments.');
+ this._installKey('A', label, this._onkeyhideall);
+
},
isAsleep: function() {
@@ -448,6 +452,15 @@
this._warnUser(pht('You must select a comment to hide.'));
},
+ _onkeyhideall: function() {
+ var inlines = this._getInlinesByType();
+ if (inlines.visible.length) {
+ this._toggleInlines('all');
+ } else {
+ this._toggleInlines('show');
+ }
+ },
+
_warnUser: function(message) {
new JX.Notification()
.setContent(message)
@@ -1698,9 +1711,15 @@
},
_onhideinlines: function(type, e) {
- this._dropdownMenu.close();
- e.prevent();
+ if (e) {
+ this._dropdownMenu.close();
+ e.prevent();
+ }
+
+ this._toggleInlines(type);
+ },
+ _toggleInlines: function(type) {
var inlines = this._getInlinesByType();
// Clear the selection state since we end up in a weird place if the

File Metadata

Mime Type
text/plain
Expires
Fri, Nov 15, 5:23 AM (2 d, 4 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6724757
Default Alt Text
D18274.id43949.diff (3 KB)

Event Timeline

Phabricator · Built by Phacility