diff --git a/modules/ve/init/mw/targets/ve.init.mw.ViewPageTarget.js b/modules/ve/init/mw/targets/ve.init.mw.ViewPageTarget.js index 4c729cbce3..0ed1d74706 100644 --- a/modules/ve/init/mw/targets/ve.init.mw.ViewPageTarget.js +++ b/modules/ve/init/mw/targets/ve.init.mw.ViewPageTarget.js @@ -455,7 +455,6 @@ ve.init.mw.ViewPageTarget.prototype.setupSkinTabs = function () { $( pVeEdit ).click( ve.bind( this.onEditTabClick, this ) ); // Allow instant switching back to view mode, without refresh $( '#ca-view a, #ca-nstab-visualeditor a' ) - .attr( 'href', '#' ) .click( ve.bind( this.onViewTabClick, this ) ); } // Source editing shouldn't highlight the edit tab @@ -619,7 +618,7 @@ ve.init.mw.ViewPageTarget.prototype.attachSaveDialog = function () { .html( ve.msg( 'minoredit' ) ); this.$saveDialog.find( '.ve-init-mw-viewPageTarget-saveDialog-watchList-label' ) .html( ve.msg( 'watchthis' ) ); - this.$saveDialog.appendTo( this.$toolbarWrapper ); + this.$toolbarWrapper.append( this.$saveDialog ); }; /** @@ -816,7 +815,8 @@ ve.init.mw.ViewPageTarget.prototype.attachToolbar = function () { this.$toolbarWrapper = this.$surface.find( '.es-toolbar-wrapper' ) .insertBefore( $( '#firstHeading' ) ) .find( '.es-toolbar' ) - .slideDown( 'fast' ); + .slideDown( 'fast' ) + .end(); }; /**