golem
/
MinervaNeue
9
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Hygiene: The main page check is not necessary 

The main page in the Minerva skin is not editable so this code is
just cruft. T89559 no longer applies.

Change-Id: Ifcd30be0d4e328b056cc7fcfb714c9783b62c913
This commit is contained in:
jdlrobson 2017-08-03 15:44:17 -07:00
parent 05734ca433
commit b478393e5e
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
resources/skins.minerva.editor/init.js
View File

@ -249,8 +249,7 @@
if ( $caEdit.find( '.edit-page' ).length === 0 ) {
$( '.nojs-edit' ).removeClass( 'nojs-edit' );
$( '#ca-edit a' ).remove();
// FIXME: unfortunately the main page is special cased.
if ( mw.config.get( 'wgIsMainPage' ) || isNewPage ||
if ( isNewPage ||
( leadSection && leadSection.text() ) || page.getSections().length === 0 ) {
// if lead section is not empty, open editor with lead section
// In some namespaces (controlled by MFNamespacesWithoutCollapsibleSections)
@ -262,10 +261,8 @@
}
}
// enable all edit pencils in sub-sections for the article namespace except for the main
// page, the pencils are unstyled there, see bug T89559
// FIXME: Merge this with the line under it after main page special handling is killed
if ( !mw.config.get( 'wgIsMainPage' ) && currentPage.getNamespaceId() === 0 ) {
// enable all edit pencils in sub-sections for the article namespace
if ( currentPage.getNamespaceId() === 0 ) {
$( '.in-block>.edit-page' ).show();
}
$( '.edit-page' ).on( 'click', function ( ev ) {