Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ext.uls.displaysettings: Update class name #434

Merged
merged 1 commit into from
Mar 5, 2022

Conversation

Krinkle
Copy link
Contributor

@Krinkle Krinkle commented Mar 5, 2022

@alistair3149
Copy link
Member

Thanks for the patch!

@alistair3149 alistair3149 merged commit 426f611 into StarCitizenTools:main Mar 5, 2022
@Krinkle Krinkle deleted the patch-1 branch March 6, 2022 00:59
wmfgerrit pushed a commit to wikimedia/mediawiki-extensions-UniversalLanguageSelector that referenced this pull request Mar 10, 2022
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
  StarCitizenTools/mediawiki-skins-Citizen#434.

Change-Id: Ia0b23d795f7807ef9227f66d0890a25524cd5a9f
wmfgerrit pushed a commit to wikimedia/mediawiki-extensions that referenced this pull request Mar 10, 2022
* Update UniversalLanguageSelector from branch 'master'
  to af6084894250c5669fb47e9bb768c45262ff9527
  - 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
      StarCitizenTools/mediawiki-skins-Citizen#434.
    
    Change-Id: Ia0b23d795f7807ef9227f66d0890a25524cd5a9f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants