From af6084894250c5669fb47e9bb768c45262ff9527 Mon Sep 17 00:00:00 2001 From: Timo Tijhof Date: Sat, 5 Mar 2022 16:17:49 +0000 Subject: [PATCH] displaysettings,inputsettings: Rename two classes for consistency I suspect these may have been accidentally mixed up or miscompleted from a nearly data-attribute or interface message, since all other classes start with `.uls-` in this repository. * "uls-input-settings-imes" is not used referenced anywhere else. * "uls-sub-panel" is referenced in one skinStyles override, updated with https://github.com/StarCitizenTools/mediawiki-skins-Citizen/pull/434. Change-Id: Ia0b23d795f7807ef9227f66d0890a25524cd5a9f --- resources/css/ext.uls.displaysettings.less | 2 +- resources/css/ext.uls.inputsettings.less | 2 +- resources/js/ext.uls.displaysettings.js | 6 +++--- resources/js/ext.uls.inputsettings.js | 4 ++-- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/resources/css/ext.uls.displaysettings.less b/resources/css/ext.uls.displaysettings.less index 368553c9..20f08110 100644 --- a/resources/css/ext.uls.displaysettings.less +++ b/resources/css/ext.uls.displaysettings.less @@ -12,7 +12,7 @@ z-index: 1; } -.ext-uls-sub-panel { +.uls-sub-panel { border-top: 1px solid #eaecf0; padding-top: 25px; top: -20px; diff --git a/resources/css/ext.uls.inputsettings.less b/resources/css/ext.uls.inputsettings.less index 9b58f143..a034d902 100644 --- a/resources/css/ext.uls.inputsettings.less +++ b/resources/css/ext.uls.inputsettings.less @@ -50,7 +50,7 @@ color: #000; } -.ext-uls-input-settings-imes-title:empty { +.uls-input-settings-imes-title:empty { display: none; } diff --git a/resources/js/ext.uls.displaysettings.js b/resources/js/ext.uls.displaysettings.js index aae888dc..d8407986 100644 --- a/resources/js/ext.uls.displaysettings.js +++ b/resources/js/ext.uls.displaysettings.js @@ -34,7 +34,7 @@ '' + // Begin display language sub-panel - '
' + + '
' + // "Display language", title above the buttons row '
' + @@ -54,7 +54,7 @@ '
' + // Begin font settings section, hidden by default - '
' + + '
' + // "Font settings" title '
' + @@ -593,7 +593,7 @@ return; } - displaySettings.$template.find( '.ext-uls-sub-panel' ).each( function () { + displaySettings.$template.find( '.uls-sub-panel' ).each( function () { var $subPanel = $( this ); // eslint-disable-next-line no-jquery/no-class-state diff --git a/resources/js/ext.uls.inputsettings.js b/resources/js/ext.uls.inputsettings.js index 09a3e67e..fe16caeb 100644 --- a/resources/js/ext.uls.inputsettings.js +++ b/resources/js/ext.uls.inputsettings.js @@ -45,7 +45,7 @@ '
' + '
' + // "Input settings for language xyz" title - '

' + + '

' + '
' + '
' + '
' + @@ -118,7 +118,7 @@ imes = $.ime.languages[ language ]; - $imeListTitle = this.$template.find( '.ext-uls-input-settings-imes-title' ); + $imeListTitle = this.$template.find( '.uls-input-settings-imes-title' ); $imeListContainer = this.$template.find( '.uls-input-settings-inputmethods-list' ); $imeListContainer.empty();