Skip to content

Update po translations from Crowdin#5593

Merged
iorsh merged 1 commit intofontforge:masterfrom
iorsh:po_update
Aug 14, 2025
Merged

Update po translations from Crowdin#5593
iorsh merged 1 commit intofontforge:masterfrom
iorsh:po_update

Conversation

@iorsh
Copy link
Copy Markdown
Contributor

@iorsh iorsh commented Aug 1, 2025

No description provided.

@iorsh iorsh requested a review from frank-trampe August 14, 2025 15:56
@frank-trampe
Copy link
Copy Markdown
Contributor

So this just fills in a few gaps that you found?

@iorsh
Copy link
Copy Markdown
Contributor Author

iorsh commented Aug 14, 2025

It's mostly the updates to Spanish translation, which was finalized, and I think this effort warrants a special update. Earlier here was a big localization PR in #5575, reviewed by @skef. It cleaned up a lot of whitespaces, which confused both translators and the Crowdin system.

Copy link
Copy Markdown
Contributor

@frank-trampe frank-trampe left a comment

Choose a reason for hiding this comment

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

This looks fine to me.

Comment thread po/es.po
msgid "DefaultFont"
msgstr "Fuente predeterminada"

msgid "DefaultShaper"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is the camel-casing (in place of a space) intentional?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, this is how all preferences look - see

{ N_("ResourceFile"), pr_file, &xdefs_filename, NULL, NULL, 'R', NULL, 0, N_("When FontForge starts up, it loads the user interface theme from\nthis file. Any changes will only take effect the next time you start FontForge.") },

Maybe I'll get to fix this eventually, but not now...

@iorsh iorsh merged commit bb8b266 into fontforge:master Aug 14, 2025
7 checks passed
@iorsh iorsh deleted the po_update branch August 17, 2025 03:45
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