Page MenuHomePhabricator

D17982.id.diff
No OneTemporary

D17982.id.diff

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' => '599698a7',
'darkconsole.pkg.js' => '1f9a31bc',
'differential.pkg.css' => '7d4cfa59',
- 'differential.pkg.js' => '886eadff',
+ 'differential.pkg.js' => '1d120743',
'diffusion.pkg.css' => 'b93d9b8c',
'diffusion.pkg.js' => '84c8f8fd',
'favicon.ico' => '30672e08',
@@ -391,8 +391,8 @@
'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' => 'cf4e2140',
- 'rsrc/js/application/diff/DiffChangesetList.js' => 'a716ca27',
- 'rsrc/js/application/diff/DiffInline.js' => '1478c3b2',
+ 'rsrc/js/application/diff/DiffChangesetList.js' => '7a184082',
+ 'rsrc/js/application/diff/DiffInline.js' => '19582231',
'rsrc/js/application/diff/ScrollObjective.js' => '7e8877e7',
'rsrc/js/application/diff/ScrollObjectiveList.js' => '6120e99a',
'rsrc/js/application/diff/behavior-preview-link.js' => '051c7832',
@@ -778,8 +778,8 @@
'phabricator-darkmessage' => 'c48cccdd',
'phabricator-dashboard-css' => 'fe5b1869',
'phabricator-diff-changeset' => 'cf4e2140',
- 'phabricator-diff-changeset-list' => 'a716ca27',
- 'phabricator-diff-inline' => '1478c3b2',
+ 'phabricator-diff-changeset-list' => '7a184082',
+ 'phabricator-diff-inline' => '19582231',
'phabricator-drag-and-drop-file-upload' => '58dea2fa',
'phabricator-draggable-list' => 'bea6e7f4',
'phabricator-fatal-config-template-css' => '8f18fa41',
@@ -981,9 +981,6 @@
'javelin-dom',
'javelin-typeahead-normalizer',
),
- '1478c3b2' => array(
- 'javelin-dom',
- ),
'1499a8cb' => array(
'javelin-behavior',
'javelin-stratcom',
@@ -1003,6 +1000,9 @@
'185bbd53' => array(
'javelin-install',
),
+ 19582231 => array(
+ 'javelin-dom',
+ ),
'19f9369b' => array(
'phui-oi-list-view-css',
),
@@ -1488,6 +1488,10 @@
'javelin-behavior',
'javelin-quicksand',
),
+ '7a184082' => array(
+ 'javelin-install',
+ 'phabricator-scroll-objective-list',
+ ),
'7a68dda3' => array(
'owners-path-editor',
'javelin-behavior',
@@ -1724,10 +1728,6 @@
'javelin-stratcom',
'javelin-dom',
),
- 'a716ca27' => array(
- 'javelin-install',
- 'phabricator-scroll-objective-list',
- ),
'a80d0378' => array(
'javelin-behavior',
'javelin-stratcom',
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
@@ -538,10 +538,24 @@
},
_setSelectionState: function(item, manager) {
- this._cursorItem = item;
+ // If we had an inline selected before, we need to update it after
+ // changing our selection to clear the selected state. Then, update the
+ // new one to add the selected state.
+ var old_inline = this.getSelectedInline();
+ this._cursorItem = item;
this._redrawSelection(manager, true);
+ var new_inline = this.getSelectedInline();
+
+ if (old_inline) {
+ old_inline.updateObjective();
+ }
+
+ if (new_inline) {
+ new_inline.updateObjective();
+ }
+
return this;
},
diff --git a/webroot/rsrc/js/application/diff/DiffInline.js b/webroot/rsrc/js/application/diff/DiffInline.js
--- a/webroot/rsrc/js/application/diff/DiffInline.js
+++ b/webroot/rsrc/js/application/diff/DiffInline.js
@@ -250,6 +250,13 @@
should_stack = true;
}
+ if (changeset.getChangesetList().getSelectedInline() === this) {
+ // TODO: Maybe add some other kind of effect here, since we're only
+ // using color to show this?
+ color = 'yellow';
+ }
+
+
objective
.setAnchor(anchor)
.setIcon(icon)

File Metadata

Mime Type
text/plain
Expires
Wed, Apr 2, 5:30 AM (3 d, 13 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7229833
Default Alt Text
D17982.id.diff (4 KB)

Event Timeline