Skip to content

Change naming of super-key-action option 'disable' to 'none'.#1974

Open
SammyMcFly wants to merge 2 commits into
pop-os:masterfrom
SammyMcFly:super-key-action-from-disabled-to-none
Open

Change naming of super-key-action option 'disable' to 'none'.#1974
SammyMcFly wants to merge 2 commits into
pop-os:masterfrom
SammyMcFly:super-key-action-from-disabled-to-none

Conversation

@SammyMcFly
Copy link
Copy Markdown
Contributor

This change was suggested/requested by @maria-komarova in the following comment: pop-os/cosmic-epoch#3087 (comment) .

Not all languages have been changed accordingly, yet, as I am not able to verify the correctness for all languages. On that note, the wording of the setting "Super key action" (en) is not perfectly consistent: in some languages, for example Spanish (es), the setting is only called "Super key" (translated into English), which the "disabled" option fits better to, than "none" does. This should probably be made consistent.

  • I have disclosed use of any AI generated code in my commit messages.
    • If you are using an LLM, and do not fully understand the changes it is making to the code base, do not create a PR.
    • In our experience, AI generated code often results in overly complex code that lacks enough context for a proper fix or feature inclusion. This results in considerably longer code reviews. Due to this, AI authored or partially authored PRs may be closed without comment.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.

@jacobgkau jacobgkau requested review from a team and removed request for a team April 21, 2026 17:18
jacobgkau
jacobgkau previously approved these changes Apr 21, 2026
Copy link
Copy Markdown
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verbiage change appears as expected, and all options for this setting still work.

@SammyMcFly
Copy link
Copy Markdown
Contributor Author

Just want to confirm, the discussion you linked was about the new Desktop -> Workspaces "Action on typing" setting, which already says "None". The change you're making here would be for the Desktop -> Window management "Super key action" setting, which currently uses "Disable".

Yes, exactly. The relevant quote in the discussion is the second part of the following: "I know Super key action was probably used as an example but we should change it there as well."

@jacobgkau
Copy link
Copy Markdown
Member

Yeah, I missed that sentence when I glanced at the page, but deleted my comment after I saw what you were referring to.

@SammyMcFly
Copy link
Copy Markdown
Contributor Author

On that note, the wording of the setting "Super key action" (en) is not perfectly consistent: in some languages, for example Spanish (es), the setting is only called "Super key" (translated into English), which the "disabled" option fits better to, than "none" does. This should probably be made consistent.

If desired, I am happy to review the languages that I am acquainted with in regards to wording.

@SammyMcFly SammyMcFly requested a review from jacobgkau May 16, 2026 21:25
@jacobgkau
Copy link
Copy Markdown
Member

If desired, I am happy to review the languages that I am acquainted with in regards to wording.

That would be appreciated, but all non-English strings need to be edited in Weblate rather than directly in the repo.

jacobgkau
jacobgkau previously approved these changes May 18, 2026
Copy link
Copy Markdown
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still looks good.

@SammyMcFly
Copy link
Copy Markdown
Contributor Author

I did changes for the translation of "Super key action" for Spanish, Italian and Russian, as their wording was not aligned with the English, as mentioned before.

When changing the selection label from "Disable" to "None", I would argue to also change the corresponding subfield of "super-key" from ".disable" to ".none" at the same time.

So, as soon as the subfield has changed and you have therefore confirmed that you commit to this naming, I will review the translation of "None", as well.

Comment thread i18n/ar/cosmic_settings.ftl Outdated
@SammyMcFly SammyMcFly force-pushed the super-key-action-from-disabled-to-none branch from aab2c92 to 5a8bdd8 Compare May 19, 2026 11:50
@SammyMcFly SammyMcFly requested review from jacobgkau and mmstick May 19, 2026 11:57
Copy link
Copy Markdown
Member

@mmstick mmstick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs extra caution when merging to ensure that Weblate is sync immediately before and immediately after merging since it removes a key.

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.

3 participants