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

Improve English #165

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

sakkamade
Copy link
Contributor

@sakkamade sakkamade commented Apr 24, 2021

This is still draft. I also haven't changed the .ts files in translation folder yet.

Wanted to understand if what I will do wouldn't be in vain. As well as have a couple of questions.

@sakkamade

This comment has been minimized.

@sakkamade sakkamade changed the title Improve English; draft 1 Improve English Apr 24, 2021
@sakkamade

This comment has been minimized.

@sakkamade sakkamade force-pushed the Improve-English branch 2 times, most recently from e54f784 to c8d132a Compare April 25, 2021 01:25
@sakkamade

This comment has been minimized.

@sakkamade

This comment has been minimized.

@sakkamade sakkamade marked this pull request as draft April 26, 2021 21:45
@sakkamade
Copy link
Contributor Author

sakkamade commented Apr 28, 2021

@kanryu, Hello!
So are you willing to review this .ui, then compare it with all 8 .ts files, which I will also change, and then build and test it?

Or it would be better if I improve only quickviewer_en.ts?

@kanryu
Copy link
Owner

kanryu commented Apr 28, 2021

@kanryu, Hello!
So are you willing to review this .ui, then compare it with all 8 .ts files, which I will also change, and then build and test it?

Basically, the translation file (* .ts) in the Qt project is automatically generated from the source code. Therefore, if you want to change the English notation from which all language resource files are derived, you should modify the source code.

If modified, the modified keywords will no longer match translations in each language. The corrections must be individually and carefully implemented.

@sakkamade
Copy link
Contributor Author

sakkamade commented Apr 28, 2021

I understand that.

Well, just to make sure, my steps at this point are:

  1. Modify .ui
  2. Replace modified strings in all .ts files as well (in some text editor)
  3. Generate new .qm files from edited .ts in Qt Linguist

Right?

</property>
</action>
<action name="actionCatalogTitleWithoutOptions">
<property name="checkable">
<bool>true</bool>
</property>
<property name="text">
<string extracomment="Ability to remove and display (parenthesized) text from Volume title in the catalog">Remove (Options) from Book Title</string>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Could you please also explain what is this supposed to do?

@kanryu
Copy link
Owner

kanryu commented Apr 29, 2021

@sakkamade Thank you for correcting the translated keywords in the source code. But if I merge your pull request, QuickViewer's translation doesn't work. This is because the translation resource side has not been updated.

To resolve this issue, lupdate Linguist in your repository. Then the translation resources will be updated.

@sakkamade
Copy link
Contributor Author

sakkamade commented Apr 29, 2021

if I merge your pull request, QuickViewer's translation doesn't work

Of course.

lupdate Linguist in your repository

I see!

But if I type

lupdate src/mainwindow.ui -ts translations/quickviewer_ar.ts

Edited strings are not carefully replaced.
What should I type exactly?

@kanryu
Copy link
Owner

kanryu commented Apr 29, 2021

Edited strings are not carefully replaced.
What should I type exactly?

This project is designed to build with qmake, and project files written with qmake can be built from Qt Creator. Modifications to your translation word will be applied to the translation file by selecting the Qt Creator menu.

edited .ui
@sakkamade
Copy link
Contributor Author

Well, I have tried to do it by Qt Creator as well. They doesn't update still.
The output lupdate warning: no TS files specified. Probably I am making some basic mistake.
It's above my head, to put it bluntly.

So I leave it as draft for ever, you may close it, use it, &c. if you want.
Sorry for taking your time.

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