-
Notifications
You must be signed in to change notification settings - Fork 483
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
Added – Turkish Language v1.0 #853
base: master
Are you sure you want to change the base?
Conversation
_data/languages.yml
Outdated
@@ -3,3 +3,4 @@ zh_CN: 简体中文 | |||
zh_TW: 繁體中文 | |||
ja: 日本語 | |||
es: Español | |||
tr: Turkish |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Türkçe?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep türkçe (eng: Turkish)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok. the word has been replaced with its own language (5c8082c)
_data/navigation.yml
Outdated
releases: | ||
title: "Versiyonlar" | ||
url: "/tr/releases/" | ||
faq: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You might have noticed, you can postpone FAQ for now,
there is a PR to remove them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the code you show is releases, not FAQ
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a dialogue created for a one line length code section
that only includes line 359, which reads: "faq". 😝
If GitHub or your browser or your mail viewer broke visuals,
that's on them. 😂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FAQ I'm postponing for now. If there is a fix, change or improvement you want, feel free to point it out. 👍
added rss pages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix – bitcoin core logo aligned to menu by sick of symmetry (d8074a0)
The property that is not in .scss lang has been removed.
Not really opposed, but mind your change from responsive. |
_data/languages.yml
Outdated
@@ -3,3 +3,4 @@ zh_CN: 简体中文 | |||
zh_TW: 繁體中文 | |||
ja: 日本語 | |||
es: Español | |||
tr: Turkish |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tr: Turkish | |
tr: Türkçe |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh of course 🤦♂️
Tested in different widths in responsive browser not found any glitch. |
The word Turkish has been replaced with its turkish equivalent (Türkçe).
Does the pic in div.site-name look good when li#langselect overflows |
I think this looks better when we compare it to the old one. But much better, I can provide a professional touch with fully responsive. |
The change will be made more professional.
pgp_key_fingerprint: "PGP anahtar parmak izi" | ||
verify_download: "İndirmenizi doğrulayın" | ||
|
||
verification_recommended: > |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder why can't I see the untranslated paragraphs that start here. 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Download page completely translated in Turkish language (131d121)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understand now, that coulda happened caused because of the
wrongly used version number fixed in 131d121 👌👌👌
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Absolutely 🤗
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indeed, just by setting the version 5 on top of that rev half translated,
the english untranslated paragraphs popped up 👌👌👌
@@ -115,7 +115,6 @@ body { | |||
a { | |||
display: block; | |||
padding: 10px 0; | |||
decoration: none; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what does this change do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The property (decoration) that is not in .css lang has been removed. I don't know why it was added either. (see. https://www.w3schools.com/cssref/)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Concept ACK,
only the Downloads page is preventing me full ACK.
Can't see the untranslated paragraphs displayed in the rendered page.
Dunno why they seemed to vanish.
Same problem in Japanese. Now, with a new update, I am making the whole page completely in Turkish, like English and Spanish. Wait for the update. |
Download page completely translated in Turkish language (131d121) |
_posts/tr/pages/2016-01-01-about.md
Outdated
type: pages | ||
layout: page | ||
lang: tr | ||
version: 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the version change you made in 131d121#r64151455
applies here and maybe to other documents.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was replaced with the version number of the English language version that I translated. (118a8de)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
5 does not exist for this document, last is version 3, see:
version: 3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
5 does not exist for this document, last is version 3, see:
version: 3
It was replaced with the version number of the English language version that I translated. (118a8de)
The original is version 5 in the English document. I converted it from version 5. |
I still don't understand why remove line: bitcoincore.org/_sass/page.scss Line 118 in fabd138
otherwise, The branch revs are untidied up, but I also think extraordinary |
Because there is no property named decoration in .css Its absence does not constitute a deficiency. Current css renderers suggest a fix to the decoration property. (see. https://www.w3schools.com/cssref/) |
Great, thanks. ACK 2508deb Thanks! |
You're welcome! I will update the header section next. |
_data/navigation.yml
Outdated
title: "Versiyonlar" | ||
url: "/tr/releases/" | ||
faq: | ||
title: "FAQs" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was removed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the info, I'll revise it now...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. Removed faq nav (8985a37)
Added – Turkish language option has been added and all pages have been translated into Turkish. ( 26dcc32 )