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

New Crowdin updates #136

Merged
merged 3 commits into from
Feb 18, 2024
Merged

New Crowdin updates #136

merged 3 commits into from
Feb 18, 2024

Conversation

FabricMCBot
Copy link
Member

No description provided.

prompt.install.successful.title=Instalado Correctamente
prompt.install.successful=Fabric Loader {0} para {1} ha sido instalado correctamente.<br>Muchos mods también necesitan que descargues <a href="{2}">Fabric API</a> en tu carpeta de <a href="fabric://mods">mods</a>.
prompt.install.successful=Fabric Loader {0} para {1} ha sido instalado correctamente.<br>Muchos mods también necesitan que descargues <a href\="{2}">Fabric API</a> en tu carpeta de <a href\="fabric\://mods">mods</a>.
Copy link
Member

Choose a reason for hiding this comment

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

Still not quite right, not sure why after I disabled escaping 🤔

Copy link
Member

Choose a reason for hiding this comment

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

Probably because : and = are key-value separators in the properties format, so it's being overly cautious.

Copy link
Member

Choose a reason for hiding this comment

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

I should just try it and see if it works as expected. Worse case I guess we could just handle these being escaped

Copy link
Member

Choose a reason for hiding this comment

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

Screenshot 2024-02-18 at 15 21 19

Works as expected, so ill merge this 👍

@modmuss50 modmuss50 merged commit a4f21b7 into master Feb 18, 2024
8 checks passed
@modmuss50 modmuss50 deleted the l10n/master branch February 18, 2024 15:22
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.

3 participants