Use veaction=edit in URL instead of action=edit when we're using multiple tabs

TODO: What about prefer-wt users switching from WTE?

Bug: T131064
Change-Id: I9aa701cbda3964decb8b1ee853c0a281567a0ccb
This commit is contained in:
Alex Monk 2016-03-30 02:44:51 +01:00
parent d4d1f10b95
commit 4210c4bee0
2 changed files with 9 additions and 3 deletions

View file

@ -175,7 +175,7 @@
$( '#ca-edit a' ).text( mw.msg( key ) );
}
if ( uri.query.action !== 'edit' ) {
if ( uri.query.action !== 'edit' && uri.query.veaction !== 'edit' ) {
if ( history.pushState ) {
// Replace the current state with one that is tagged as ours, to prevent the
// back button from breaking when used to exit VE. FIXME: there should be a better
@ -301,7 +301,10 @@
);
// On a view page, extend the current URI so parameters like oldid are carried over
// On a non-view page, use viewUri
if ( conf.singleEditTab ) {
if (
conf.singleEditTab &&
mw.user.options.get( 'visualeditor-tabs' ) !== 'multi-tab'
) {
veEditUri = viewUri.clone().extend( { action: 'edit' } );
delete veEditUri.query.veaction;
} else {

View file

@ -1114,7 +1114,10 @@ ve.init.mw.DesktopArticleTarget.prototype.transformPage = function () {
// Set the current URL
uri = this.currentUri;
if ( mw.config.get( 'wgVisualEditorConfig' ).singleEditTab ) {
if (
mw.config.get( 'wgVisualEditorConfig' ).singleEditTab &&
mw.user.options.get( 'visualeditor-tabs' ) !== 'multi-tab'
) {
uri.query.action = 'edit';
mw.config.set( 'wgAction', 'edit' );
} else {