Skip to content
This repository has been archived by the owner on Jul 22, 2023. It is now read-only.

Update _i18n.scss #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

code-a-man
Copy link

Update tr to be a better translation.

Also, It could be a better optional option to put the "jump-to" variable after content for additive languages like Turkish.
For example:

`${jumptoprefix}${content}${jumptosuffix}` //example placeholder

const en = {
jumptoprefix: "Jump to ", // Jump to Discord
jumptosuffix: ""
}

const tr = {
jumptoprefix: "",
jumptosuffix: "'a atla" // Discord'a atla
}

In this example, you just type what you need prefix or suffix.
I'm not capable of scss rn so I gave the example on js. It is not that good example but I hope I explained what I mean.

If you don't agree my suggestion just merging this pr will greatly improve the Turkish translation.

Update tr to be a better translation.
Also, It could be a better optional option to put the "jump-to" variable after content for additive languages like Turkish.
@CorellanStoma
Copy link
Member

CorellanStoma commented Jul 16, 2022

Hmm I think I understand what you mean when I have more free time for the theme projects, I will try it out.

Thanks for your feedback!

@CorellanStoma
Copy link
Member

I am currently revising the theme add-on, with the next update I will include a prefix and suffix variable. In addition, a distinction is made between internal and external
code2
.

@code-a-man
Copy link
Author

Making internal and external is an amazing idea. After revising, I will update Turkish translation too.

@CorellanStoma
Copy link
Member

Thanks for that, I only speak German and English, I do use DeepL, but sometimes the translations are not optimal.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants