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] File Dialog Save and Cancel Buttons not translated #2474

Closed
2 of 5 tasks
tscheims1 opened this issue Sep 17, 2021 · 5 comments
Closed
2 of 5 tasks

[Bug] File Dialog Save and Cancel Buttons not translated #2474

tscheims1 opened this issue Sep 17, 2021 · 5 comments
Labels
bug A bug that affects the functionality of Zettlr. external Issue This is an issue caused by a third-party library or integration.

Comments

@tscheims1
Copy link

Description

When I save a File the "save" and "cancel" button of the File Dialog are not translated. (the buttons are named "gtk-save" and "gtk-cancel")

Reproducing

1.) Create new File
2.) Save the file
3:) File Dialog opened

Zettlr Version

Beta (if applicable)

Specify version

Zettlr 2 Beta 4

Your Platform

  • Windows
  • macOS
  • Linux

Architecture

  • Intel 64bit
  • ARM 64bit

Operating System Version

Manjaro Linux, i3, packages up to date.

Additional Information

It's a minor /low priority issue.

@tscheims1 tscheims1 added the bug A bug that affects the functionality of Zettlr. label Sep 17, 2021
@nathanlesage
Copy link
Member

This almost sounds like a bug in the Electron framework. What Zettlr does is simply call the file-saving dialog which is offered by Electron, and we do not override any button labels in there. Plus the gtk--prefix indicates that it looks like the placeholder by the GTK window manager. Maybe it's just a bug in the 14.0.0 release of Electron, I've seen there is a newer one. Let me quickly check … Okay the 14.0.1 apparently does not fix anything related to that.

Just to check, would you please mind checking the first beta version of Zettlr 2.0 to see if the buttons are translated there? This way we might be able to pinpoint it down to Electron 14.

@tscheims1
Copy link
Author

tscheims1 commented Sep 17, 2021

@nathanlesage Downloaded first Beta: Jup you re right the buttons are correct translated.

@nathanlesage
Copy link
Member

Then it pretty much sounds like a bug in the electron framework! Would you mind opening up a bug with them (but only if it hasn't yet been reported!)? Their repo is here: https://github.com/electron/electron

You can tell them that the last working electron version was 13.1.9 and that we see the bug with electron 14.0.0.

That would be awesome!

@nathanlesage nathanlesage added the external Issue This is an issue caused by a third-party library or integration. label Sep 17, 2021
@tscheims1
Copy link
Author

@nathanlesage It's already reported: electron/electron#30247
and will be fixed soon (pull request is waiting for merge)
electron/electron#30888

@nathanlesage
Copy link
Member

Ah peeeeerfect! :) Then I'll be closing this issue here, because I always update any dependencies as soon as there's a new version available, so as soon as Electron fixes this bug, Zettlr will follow suit! Thank you for looking it up!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug that affects the functionality of Zettlr. external Issue This is an issue caused by a third-party library or integration.
Projects
None yet
Development

No branches or pull requests

2 participants