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

desktop: Internationalize file pickers & improve filters #17604

Merged
merged 2 commits into from
Sep 11, 2024

Conversation

kjarosh
Copy link
Member

@kjarosh kjarosh commented Aug 21, 2024

This PR adds internationalization to strings in file pickers and improves a bit their filters.

image

@kjarosh kjarosh added the A-desktop Area: Desktop Application label Aug 21, 2024
Comment on lines 6 to 8
picker-filter-spl = FutureSplash Animator (*.spl)
picker-filter-ruf = Ruffle Bundle (*.ruf)
Copy link
Contributor

Choose a reason for hiding this comment

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

FutureSplash Animator probably can't be translated but could perhaps be transliterated

Ruffle Bundle could be translated ("Ruffle" should not, but "Bundle" I think could be)

Comment on lines 6 to 8
picker-filter-spl = FutureSplash Animator (*.spl)
picker-filter-ruf = Ruffle Bundle (*.ruf)
Copy link
Member

Choose a reason for hiding this comment

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

I wonder - what's the chance that these indeed can be translated in any meaningful way to something else than what they currently are? And even if so, do we want that?

Copy link
Contributor

Choose a reason for hiding this comment

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

Did we just do the time warp?

Copy link
Member

Choose a reason for hiding this comment

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

Sorry, I recreated the note right when you replied to it... Because it seems the GH UI is buggy - this was meant to also correspond to SWF (.swf) above...

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, to me, even "bundle" is near that territory of "jargon" words that are not meant to be messed with... Like "shader" for example.

Copy link
Member Author

Choose a reason for hiding this comment

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

I would expect that in most languages they will be the same as in English, however I added entries for them too for the sake of consistency and in case some language will require a translation (e.g. by adding a "file" suffix to SWF because it would be ambiguous otherwise — I don't know, just guessing, but I would have to know every language to make sure it's not needed). Looking for instance at Inkscape, some words might be translatable ("bundle" perhaps?):

@kjarosh kjarosh force-pushed the picker-i18n branch 2 times, most recently from e6d8af1 to 6c85edc Compare August 22, 2024 22:41
@torokati44
Copy link
Member

For the record, I'm neutral about this change. Not approving, but not blocking either. 🤷‍♂️

@kjarosh kjarosh added the waiting-on-review Waiting on review from a Ruffle team member label Aug 26, 2024
@kjarosh kjarosh force-pushed the picker-i18n branch 3 times, most recently from be44ccf to ea56551 Compare September 1, 2024 06:38
Copy link
Member

@sleepycatcoding sleepycatcoding left a comment

Choose a reason for hiding this comment

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

Apart from my one comment, LGTM!

desktop/src/gui/picker.rs Outdated Show resolved Hide resolved
@torokati44 torokati44 enabled auto-merge (rebase) September 11, 2024 11:25
@torokati44 torokati44 merged commit 75e4eaa into ruffle-rs:master Sep 11, 2024
16 of 17 checks passed
@kjarosh kjarosh deleted the picker-i18n branch September 11, 2024 11:48
@sleepycatcoding sleepycatcoding removed the waiting-on-review Waiting on review from a Ruffle team member label Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-desktop Area: Desktop Application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants