refactor(core): ♻️ use the same code pattern for SkinEditSectionLinks

This commit is contained in:
alistair3149 2022-11-15 22:39:10 -05:00
parent 2ec7da29a5
commit 5c375bb71d
No known key found for this signature in database

View file

@ -116,21 +116,24 @@ class SkinHooks implements
* @param Language $lang
*/
public function onSkinEditSectionLinks( $skin, $title, $section, $sectionTitle, &$result, $lang ) {
// Be extra safe because it might be active on other skins with caching
if ( $skin->getSkinName() === 'citizen' && $result ) {
/*
* FIXME: For some reason if you modify the VE button, it duplicates one automatically
* Bug: T323186
*
* if ( $result['veeditsection'] ) {
* if ( isset( $result['veeditsection'] ) ) {
* $result['veeditsection']['attribs']['class'] = 'mw-ui-icon-wikimedia-edit';
* }
*/
// Add icon to edit section link
// If VE button is present, use wikiText icon
if ( $result['editsection'] ) {
if ( isset( $result['editsection'] ) ) {
$result['editsection']['attribs']['class'] = $result['veeditsection'] ? 'mw-ui-icon-wikimedia-wikiText' : 'mw-ui-icon-wikimedia-edit';
}
}
}
/**
* SkinPageReadyConfig hook handler