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

Added – Turkish Language v1.0 #853

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

zackha
Copy link

@zackha zackha commented Jan 18, 2022

Added – Turkish language option has been added and all pages have been translated into Turkish. ( 26dcc32 )

@@ -3,3 +3,4 @@ zh_CN: 简体中文
zh_TW: 繁體中文
ja: 日本語
es: Español
tr: Turkish
Copy link
Contributor

Choose a reason for hiding this comment

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

Türkçe?

Copy link
Author

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)

Copy link
Author

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)

releases:
title: "Versiyonlar"
url: "/tr/releases/"
faq:
Copy link
Contributor

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.

Copy link
Author

@zackha zackha Jan 18, 2022

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

Copy link
Contributor

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. 😂

Copy link
Author

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. 👍

Copy link
Author

@zackha zackha left a 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.
@zackha zackha changed the title added – turkish language v1 added – turkish language and a few fixes v1 Jan 18, 2022
@katesalazar
Copy link
Contributor

Fix – bitcoin core logo aligned to menu by sick of symmetry (d8074a0)

Not really opposed, but mind your change from responsive.
Have you tested well different page widths?

@@ -3,3 +3,4 @@ zh_CN: 简体中文
zh_TW: 繁體中文
ja: 日本語
es: Español
tr: Turkish
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
tr: Turkish
tr: Türkçe

Copy link
Author

Choose a reason for hiding this comment

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

oh of course 🤦‍♂️

@zackha
Copy link
Author

zackha commented Jan 18, 2022

Fix – bitcoin core logo aligned to menu by sick of symmetry (d8074a0)

Not really opposed, but mind your change from responsive. Have you tested well different page widths?

Tested in different widths in responsive browser not found any glitch.

The word Turkish has been replaced with its turkish equivalent (Türkçe).
@katesalazar
Copy link
Contributor

...

Not really opposed, but mind your change from responsive. Have you tested well different page widths?

Tested in different widths in responsive browser and found not found any glitch.

Does the pic in div.site-name look good when li#langselect overflows
to the second row, under the "About" li?
In my browser that's a questionable alignment.
It's the only width that I've found worse for your change, though.

@zackha
Copy link
Author

zackha commented Jan 18, 2022

...

Not really opposed, but mind your change from responsive. Have you tested well different page widths?

Tested in different widths in responsive browser and found not found any glitch.

Does the pic in div.site-name look good when li#langselect overflows to the second row, under the "About" li? In my browser that's a questionable alignment. It's the only width that I've found worse for your change, though.

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: >
Copy link
Contributor

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. 🤔

Copy link
Author

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)

Copy link
Contributor

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 👌👌👌

Copy link
Author

Choose a reason for hiding this comment

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

Absolutely 🤗

Copy link
Contributor

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;
Copy link
Contributor

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?

Copy link
Author

@zackha zackha Jan 19, 2022

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/)

Copy link
Contributor

@katesalazar katesalazar left a 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.

@zackha zackha closed this Jan 19, 2022
@zackha zackha deleted the stable branch January 19, 2022 07:13
@zackha zackha restored the stable branch January 19, 2022 07:14
@zackha
Copy link
Author

zackha commented Jan 19, 2022

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.

@zackha zackha reopened this Jan 19, 2022
@zackha
Copy link
Author

zackha commented Jan 19, 2022

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.

Download page completely translated in Turkish language (131d121)

type: pages
layout: page
lang: tr
version: 1
Copy link
Contributor

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.

Copy link
Author

@zackha zackha Jan 19, 2022

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)

Copy link
Contributor

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:

Copy link
Author

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:

It was replaced with the version number of the English language version that I translated. (118a8de)

@zackha
Copy link
Author

zackha commented Jan 19, 2022

why did you change this? (it was previously wrongly at 1)

The original is version 5 in the English document. I converted it from version 5.

@katesalazar
Copy link
Contributor

katesalazar commented Jan 19, 2022

I still don't understand why remove line:

decoration: none;

otherwise,
ACK 2508deb

The branch revs are untidied up, but I also think extraordinary
and urgency circumstances concur justifying a looser merge
requirements for this particular PR

@zackha
Copy link
Author

zackha commented Jan 19, 2022

I still don't understand why remove line:

decoration: none;

otherwise, ACK 2508deb

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/)

@katesalazar
Copy link
Contributor

Great, thanks.

ACK 2508deb

Thanks!

@zackha
Copy link
Author

zackha commented Jan 19, 2022

Great, thanks.

ACK 2508deb

Thanks!

You're welcome! I will update the header section next.

@zackha zackha changed the title added – turkish language and a few fixes v1 Added – Turkish Language v1.0 Jan 23, 2022
title: "Versiyonlar"
url: "/tr/releases/"
faq:
title: "FAQs"
Copy link

Choose a reason for hiding this comment

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

this was removed

Copy link
Author

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...

Copy link
Author

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)

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