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

[Bug]: Button labels in File choosers are untranslated on Linux in e14 #30247

Closed
3 tasks done
Kilian opened this issue Jul 23, 2021 · 9 comments · Fixed by #30888
Closed
3 tasks done

[Bug]: Button labels in File choosers are untranslated on Linux in e14 #30247

Kilian opened this issue Jul 23, 2021 · 9 comments · Fixed by #30888
Assignees
Labels
14-x-y bug 🪲 has-repro-gist Issue can be reproduced with code at https://gist.github.com/ platform/linux

Comments

@Kilian
Copy link
Member

Kilian commented Jul 23, 2021

Preflight Checklist

Electron Version

14.0.0

What operating system are you using?

Ubuntu

Operating System Version

Ubuntu 18.04

What arch are you using?

x64

Last Known Working Electron version

13.1.7

Expected Behavior

The buttons in a (native) file dialog read "Cancel" and "OK"

Actual Behavior

The buttons in a native file dialog read gtk-cancel and gtk-ok.

Testcase Gist URL

https://gist.github.com/Kilian/3de5f404d90d9b12ccaec817fdc4d6da

Additional Information

Main window has a file input, click it to open the File chooser. Test on Ubuntu between 14.x and 13.x to see the difference. Probably caused by #19159

Electron 13.1.7:
Selection_999(158)

Electron 14.0.0-beta.16
Selection_999(156)

@Kilian Kilian changed the title [Bug]: [Bug]: Button labels in File choosers are untranslated on Linux in e14 Jul 23, 2021
@clavin clavin added the has-repro-gist Issue can be reproduced with code at https://gist.github.com/ label Aug 2, 2021
@tristan957
Copy link
Contributor

Could you tell me how to run the repro gist and I can take a look?

@tristan957
Copy link
Contributor

One thing to test in your environment is running the example with GTK_USE_PORTAL=1 and not setting the variable at all and seeing if that makes a difference for you.

@tristan957
Copy link
Contributor

@Kilian
Copy link
Member Author

Kilian commented Aug 6, 2021

Could you tell me how to run the repro gist and I can take a look?

@tristan957 You can open it in fiddle!

@tristan957
Copy link
Contributor

tristan957 commented Sep 1, 2021

It seems like a mistake to have shipped with this bug. I have no clue how Electron internationalization works. If someone could inform me how to get a translated string out of the electron infrastructure, I could possibly fix this. Until then, #19159 should be reverted.

@tristan957
Copy link
Contributor

From #gtk with regard to using GTK_STOCK_*...

Well. Step one is just stop. Step two is they need to do translations. No way around it

@codebytere codebytere self-assigned this Sep 8, 2021
@codebytere
Copy link
Member

@tristan957 working on a fix for this which should be up within a few days.

@tristan957
Copy link
Contributor

@codebytere would appreciate a ping when you have a PR up just so I can learn for myself what the proper fix is.

@codebytere
Copy link
Member

@tristan957 just opened it at #30888

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
14-x-y bug 🪲 has-repro-gist Issue can be reproduced with code at https://gist.github.com/ platform/linux
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants