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' => '3ac6e174', 'darkconsole.pkg.js' => '1f9a31bc', 'differential.pkg.css' => '113e692c', - 'differential.pkg.js' => '5d53d5ce', + 'differential.pkg.js' => 'f6d809c0', 'diffusion.pkg.css' => 'a2d17c7d', 'diffusion.pkg.js' => '6134c5a1', 'favicon.ico' => '30672e08', @@ -396,7 +396,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' => 'b49b59d6', - 'rsrc/js/application/diff/DiffChangesetList.js' => '1f2e5265', + 'rsrc/js/application/diff/DiffChangesetList.js' => 'e74b7517', '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', @@ -776,7 +776,7 @@ 'phabricator-darkmessage' => 'c48cccdd', 'phabricator-dashboard-css' => 'fe5b1869', 'phabricator-diff-changeset' => 'b49b59d6', - 'phabricator-diff-changeset-list' => '1f2e5265', + 'phabricator-diff-changeset-list' => 'e74b7517', 'phabricator-diff-inline' => 'e83d28f3', 'phabricator-drag-and-drop-file-upload' => '58dea2fa', 'phabricator-draggable-list' => 'bea6e7f4', @@ -1044,10 +1044,6 @@ 'javelin-uri', 'javelin-routable', ), - '1f2e5265' => array( - 'javelin-install', - 'phuix-button-view', - ), '1f6794f6' => array( 'javelin-behavior', 'javelin-stratcom', @@ -2093,6 +2089,10 @@ 'javelin-workflow', 'javelin-magical-init', ), + 'e74b7517' => array( + 'javelin-install', + 'phuix-button-view', + ), 'e83d28f3' => array( 'javelin-dom', ), 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 @@ -128,6 +128,9 @@ this._redrawFocus(); this._redrawSelection(); this.resetHover(); + + this._bannerChangeset = null; + this._redrawBanner(); }, wake: function() { @@ -136,6 +139,9 @@ this._redrawFocus(); this._redrawSelection(); + this._bannerChangeset = null; + this._redrawBanner(); + if (this._initialized) { return; } @@ -1374,6 +1380,11 @@ var node = this._getBannerNode(); var changeset = this._getVisibleChangeset(); + if (!changeset) { + JX.DOM.remove(node); + return; + } + // Don't do anything if nothing has changed. This seems to avoid some // flickering issues in Safari, at least. if (this._bannerChangeset === changeset) { @@ -1381,11 +1392,6 @@ } this._bannerChangeset = changeset; - if (!changeset) { - JX.DOM.remove(node); - return; - } - var inlines = this._getInlinesByType(); var unsaved = inlines.unsaved;