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

Internationalization and keyboard shotcuts SHIFT #66

Closed
linuxeng opened this issue Jan 8, 2018 · 6 comments
Closed

Internationalization and keyboard shotcuts SHIFT #66

linuxeng opened this issue Jan 8, 2018 · 6 comments

Comments

@linuxeng
Copy link

linuxeng commented Jan 8, 2018

Good Day.
I assembled the program and ran into the fact that the path /usr/share/flameshot/translations/ is constant. When compile with default prefix /usr/local. ,

QTranslator translator;
    translator.load(QLocale::system().language(),
      "Internationalization", "_", "/usr/share/flameshot/translations/");

Because of this, when compile with default prefix /usr/local the localization does not work and you have to manually create a link or copy translations into / usr / share /.
See file src/main.cpp

Shotcuts SHIFT to the righ. and SHIFT bottom does not work.

Best Regards.
Vadim K.
Russia.

@lupoDharkael
Copy link
Collaborator

It's true, translations are broken now. The code needs a change because I'm working on the Windows port and that implies completely different paths.

Thanks for the report.

@lupoDharkael
Copy link
Collaborator

Now translations use a fixed install path again. It should work as before while I rewrite it for windows and I implement non fixed paths in the code.

@lupoDharkael
Copy link
Collaborator

The shortcuts should work now.

@pafri
Copy link
Contributor

pafri commented Jan 4, 2019

Czech localization file. Attached, ready to be committed. Best regards. Pavel Fric
Internationalization_cs.zip

@hosiet
Copy link
Member

hosiet commented Jan 6, 2019

@pafri I guess you should provide it in the form of Pull Request or at least please open a new issue. It doesn't make sense to provide translation files in a totally irrelevant issue.

@pafri
Copy link
Contributor

pafri commented Jan 7, 2019 via email

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

No branches or pull requests

4 participants