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.

Sign up for GitHub

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

Jump to bottom

NSIS: Fix Turkish translation #19177

Closed
wants to merge 1 commit into from
Closed

NSIS: Fix Turkish translation #19177

wants to merge 1 commit into from

Conversation

Gl1tchs
Copy link

@Gl1tchs Gl1tchs commented Jun 16, 2023

I fixed some punctuation and grammatical errors with some logical translation errors.

@thalieht thalieht added Translations Related to i18n or Transifex NSIS NSIS installer related labels Jun 16, 2023
@glassez glassez changed the title Typo and grammar correction on turkish.nsi NSIS: Fix Turkish translation Jun 18, 2023
@glassez glassez requested a review from a team June 18, 2023 05:39
@glassez glassez added this to the 4.6.0 milestone Jun 18, 2023
@BouRock
Copy link
Contributor

BouRock commented Jun 18, 2023 •
edited

"Create" word needs to be started with lowercase. Because there must be a translation visuality.

${LANG_TURKISH} "Masaüstü Kısayolu oluştur"
${LANG_TURKISH} "Başlangıç Menüsü Kısayolu oluştur"

So why the followings' first word started with lowercase in translations:
"Start ... ", "Open .... ", "Add .... ", "Disable .... ", etc."

${LANG_TURKISH} "Windows başlangıcında qBittorrent'i başlat"
${LANG_TURKISH} ".torrent dosyalarını qBittorrent ile aç"
${LANG_TURKISH} "Magnet bağlantılarını qBittorrent ile aç"
${LANG_TURKISH} "Windows Güvenlik Duvarı kuralı ekle"
${LANG_TURKISH} "Windows yol uzunluğu sınırını etkisizleştir (260 karakter MAX_PATH sınırlaması, Windows 10 1607 veya sonrasını gerektirir)"

@Gl1tchs
Copy link
Author

Gl1tchs commented Jun 22, 2023 •
edited

Well I noticed that but I saw that one word is uppercase and the other word is not that would seem odd.

@glassez glassez mentioned this pull request Jul 5, 2023
Merged
@glassez
Copy link
Member

glassez commented Jul 5, 2023

Superseded by #19242.

@glassez glassez closed this Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
NSIS NSIS installer related Translations Related to i18n or Transifex
Projects
None yet
Milestone
4.6.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Gl1tchs @BouRock @glassez @thalieht