Merge "Remove some useless code trying to show things that are always visible"

This commit is contained in:
jenkins-bot 2018-09-26 20:18:59 +00:00 committed by Gerrit Code Review
commit ab593a3268
1 changed files with 1 additions and 10 deletions

View File

@ -95,9 +95,7 @@
function updateEditPageButton( enabled ) {
$caEdit
.addClass( enabled ? enabledClass : disabledClass )
.removeClass( enabled ? disabledClass : enabledClass )
// TODO: can hidden be removed from the default state?
.removeClass( 'hidden' );
.removeClass( enabled ? disabledClass : enabledClass );
}
/**
@ -271,8 +269,6 @@
}
} );
updateEditPageButton( true );
// reveal edit links on user pages
page.$( '.edit-link' ).removeClass( 'hidden' );
currentPage.getRedLinks().on( 'click', function ( ev ) {
var drawerOptions = {
progressiveButton: new Button( {
@ -313,11 +309,6 @@
}
}
// enable all edit pencils in sub-sections for the article namespace
if ( currentPage.getNamespaceId() === 0 ) {
$( '.in-block>.edit-page' ).show();
}
if ( !router.getPath() && ( mw.util.getParamValue( 'veaction' ) || mw.util.getParamValue( 'action' ) === 'edit' ) ) {
if ( mw.util.getParamValue( 'veaction' ) === 'edit' ) {
editorOverride = 'VisualEditor';