Skip to content

Conversation

@onesounds
Copy link
Contributor

@onesounds onesounds commented Feb 10, 2023

What's the PR

  • Fix PROPOSAL. Language file. Singular/Plural #1888

  • Change Hotkey/Plugin title area to plural (Hotkeys / plugins)

  • Fix Hotkey tab's singular texts : hotkeys / shortcuts

  • Other languages have added key values and pasted existing text.

  • 'Themes' To 'Appearance'

    • Added a key to avoid confusion in translation.

image

image

image

Test Case

  • There should be no miss typos in the key value.

@onesounds onesounds self-assigned this Feb 10, 2023
@onesounds onesounds changed the title Change Singular Hotkey/Plugin title area to Plural (Hotkeys/Plugins) Change Singular Hotkey/Plugin texts to Plural (Hotkeys/Plugins) Feb 10, 2023
@deefrawley
Copy link
Member

deefrawley commented Feb 10, 2023

Did we want to add the change 'Themes' tab to 'Appearance' to this PR as well?

@onesounds
Copy link
Contributor Author

onesounds commented Feb 10, 2023

Did we want to add the change 'Themes' tab to 'Appearance' to this PR as well?

I forgot. I'll reflect it in this pr.

@onesounds onesounds changed the title Change Singular Hotkey/Plugin texts to Plural (Hotkeys/Plugins) Adjust sidebar texts in setting window Feb 10, 2023
<system:String x:Key="searchplugin_Noresult_Title">No results found</system:String>
<system:String x:Key="searchplugin_Noresult_Subtitle">Please try a different search.</system:String>
<system:String x:Key="plugin">Plugin</system:String>
<system:String x:Key="plugins">Plugins</system:String>
Copy link
Member

Choose a reason for hiding this comment

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

Just update en.xaml should be fine. This is fairly annoying to update all files

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You will not understand this. People with low IQs usually work like this.

Copy link
Member

@taooceros taooceros Feb 11, 2023

Choose a reason for hiding this comment

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

Na I just want to inform you the fact so next time you don't have to do it🤣

Copy link
Contributor Author

@onesounds onesounds Feb 11, 2023

Choose a reason for hiding this comment

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

kidding, actaully I also knew that adding an English key would reflect other languages.
but anyway If I follow you, doesn't every other language display "plugins(english)"?
I don't want to affect other languages, so I copied all the 'plugin' values into the 'plugins'.

image

If I didn't want to influence other languages, isn't this the right way to do it?

Copy link
Member

@taooceros taooceros Feb 11, 2023

Choose a reason for hiding this comment

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

The issue is crowdin won't automatically load translation (which means unless we do a manually upload translation (which sometimes introduce weird thing), the update won't work and will be replaced in next crowdin merge)....and I think it's ok to let translator revises the string if we change the source string. Sometimes crowdin can detect the change from plugin->plugins and will revise the translation in other language as well.

Copy link
Contributor

Choose a reason for hiding this comment

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

So should we revert non-English changes?

Copy link
Member

@taooceros taooceros Feb 20, 2023

Choose a reason for hiding this comment

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

No no need for that. I am just saying it won't be flushed into crowdin automatically

@VictoriousRaptor VictoriousRaptor merged commit b2516ed into Flow-Launcher:dev Feb 21, 2023
@jjw24 jjw24 added the bug Something isn't working label Feb 21, 2023
@jjw24 jjw24 added this to the 1.14.0 milestone Feb 21, 2023
@jjw24 jjw24 mentioned this pull request Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working kind/i18n

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PROPOSAL. Language file. Singular/Plural

5 participants