Skip to content

Strings rework#227

Merged
Olf0 merged 51 commits intosailfishos-patches:masterfrom
Olf0:Olf0-strings-rework
Jan 13, 2022
Merged

Strings rework#227
Olf0 merged 51 commits intosailfishos-patches:masterfrom
Olf0:Olf0-strings-rework

Conversation

@Olf0
Copy link
Contributor

@Olf0 Olf0 commented Jan 11, 2022

Adapt strings to nomenclature at https://github.com/sailfishos-patches/patchmanager/wiki/Terms-and-wording
Also fix largely broken SettingsPage.qml section in settings-patchmanager.ts: Many incorrect line numbers and some missing entries.

@Olf0 Olf0 marked this pull request as ready for review January 11, 2022 23:07
@Olf0 Olf0 added bug something that needs fixing translation source strings, TS files, Transifex integration etc. labels Jan 11, 2022
@Olf0 Olf0 added this to the 3.2.1 milestone Jan 11, 2022
@Olf0 Olf0 requested a review from nephros January 11, 2022 23:31
@Olf0 Olf0 self-assigned this Jan 11, 2022
@Olf0
Copy link
Contributor Author

Olf0 commented Jan 11, 2022

Is there a (semi-) automatic way to check, that I did not break some strings?
E.g., running lrelease on the source files in this branch and comparing its output to the settings-patchmanager.ts I submitted in this branch?

@Olf0 Olf0 added the UX design and behaviour of UI components label Jan 12, 2022
Copy link
Contributor

@nephros nephros left a comment

Choose a reason for hiding this comment

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

Wow, nice.

Looking forward to the next wave of translation merge requests!

@Olf0 Olf0 merged commit f0f487b into sailfishos-patches:master Jan 13, 2022
@Olf0
Copy link
Contributor Author

Olf0 commented Jan 13, 2022

Looking forward to the next wave of translation merge requests!

This is automatically triggered now.

Still I have to go through the README to adapt it to the new wording … some day.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug something that needs fixing translation source strings, TS files, Transifex integration etc. UX design and behaviour of UI components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Usability bug: Inconsistent wording] "install*" instead of "apply*" and "remove" instead of "unapply" used in notifications

2 participants