Skip to content

[DOCS-DE] rewrite setup index page#17966

Merged
k8s-ci-robot merged 3 commits intokubernetes:masterfrom
eumel8:issue_17865
Jan 17, 2020
Merged

[DOCS-DE] rewrite setup index page#17966
k8s-ci-robot merged 3 commits intokubernetes:masterfrom
eumel8:issue_17865

Conversation

@eumel8
Copy link
Copy Markdown
Contributor

@eumel8 eumel8 commented Dec 5, 2019

fixes #17865

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 5, 2019
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

Welcome @eumel8!

It looks like this is your first PR to kubernetes/website 🎉. Please refer to our pull request process documentation to help your PR have a smooth ride to approval.

You will be prompted by a bot to use commands during the review process. Do not be afraid to follow the prompts! It is okay to experiment. Here is the bot commands documentation.

You can also check if kubernetes/website has its own contribution guidelines.

You may want to refer to our testing guide if you run into trouble with your tests not passing.

If you are having difficulty getting your pull request seen, please follow the recommended escalation practices. Also, for tips and tricks in the contribution process you may want to read the Kubernetes contributor cheat sheet. We want to make sure your contribution gets all the attention it needs!

Thank you, and welcome to Kubernetes. 😃

@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Dec 5, 2019
@k8s-ci-robot k8s-ci-robot added language/de Issues or PRs related to German language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Dec 5, 2019
@netlify
Copy link
Copy Markdown

netlify bot commented Dec 5, 2019

Deploy preview for kubernetes-io-master-staging ready!

Built with commit a2ba065

https://deploy-preview-17966--kubernetes-io-master-staging.netlify.com


Überlegen Sie sich bei der Bewertung einer Lösung für eine Produktionsumgebung, welche Aspekte des Betriebs eines Kubernetes-Clusters (oder von _abstractions_) Sie selbst verwalten oder an einen Anbieter auslagern möchten.

Einige mögliche Abstraktionen von Kubernetes-Clustern sind {{< glossary_tooltip text="applications" term_id="applications" >}}, {{< glossary_tooltip text="data plane" term_id="data-plane" >}}, {{< glossary_tooltip text="control plane" term_id="control-plane" >}}, {{< glossary_tooltip text="cluster infrastructure" term_id="cluster-infrastructure" >}} und {{< glossary_tooltip text="cluster operations" term_id="cluster-operations" >}}.
Copy link
Copy Markdown
Contributor

@sftim sftim Dec 8, 2019

Choose a reason for hiding this comment

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

The tooltip display text is not translated here. Is that intentional?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

related to the svg description in English, so the user can easier find the connections


Die folgende Tabelle für Produktionsumgebungs-Lösungen listet Anbieter und deren Lösungen auf.

|Providers | Managed | Turnkey cloud | On-Prem Datacenter | Custom (cloud) | Custom (On-premises VMs)| Custom (Bare Metal) |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These headings are not translated. Is that intentional?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I left it as it is (tooltip display text and table headings). If you look at the test output [1] this text corresponds with the content in the image. We can build our own SVG in German but I think thats not easy to maintain in the future.

[1] https://deploy-preview-17966--kubernetes-io-master-staging.netlify.com/de/docs/setup/

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ah OK. It would be better to localize that image; ideally, there would be tooling to make that easy (right now, there is not).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I investigated it a little bit. It's possible to extend the SVG in multi-language style

<switch>
    <text
       systemLanguage="de"
       german text
     </text>
    <text
       systemLanguage="en"
       english text
     </text>
</switch>

But that would be a very big file in multi-lang and length and size must be adjusted for each text string. Another way would be to use a markdown table instead SVG. Otherwise in Japanese and Korean are already used the origin, maybe it's also a way for German :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hi @sftim,
I want to circle back to this PR. After one month it seems I had no time to work on this SVG stuff. The technical solution should be clear, but I'm not sure if this is common between all language teams. My little hope would be to merge this PR at it is (it's also my first one) and work on the SVG topic later. What do you think?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

That's fine. I hope the German-language localization team agree.
It is usually quite trick to localize images; it's an ideal and in my view it is fine to merge PRs that leave images not yet localized.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I am ok with keeping the SVG in English. @rlenferink @remyleone what do you think?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would say let's keep them in English as it is easier to maintain. But the German localization team is the one that will have the final say imho

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd say keep the SVG in English because (as Remy said) it's indeed easier to maintain.

@rlenferink
Copy link
Copy Markdown
Member

How about the actual content change of this PR @bene2k1 ?

@bene2k1
Copy link
Copy Markdown
Member

bene2k1 commented Jan 17, 2020

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 17, 2020
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bene2k1

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 17, 2020
@k8s-ci-robot k8s-ci-robot merged commit 1226970 into kubernetes:master Jan 17, 2020
@eumel8 eumel8 deleted the issue_17865 branch January 17, 2020 10:15
wawa0210 pushed a commit to wawa0210/website that referenced this pull request Mar 2, 2020
* [DOCS-DE] rewrite setup index page

* fix table caption

* fix whitespace
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/de Issues or PRs related to German language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue with k8s.io/de/docs/setup/

6 participants