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

i18n(de): update markdown-content.mdx #8037

Open
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

DerTimonius
Copy link
Contributor

Description (required)

This PR updates the german translation of the markdown-content.mdx documentation. I initially wanted to start with the integration guide for mdx when I noticed that it references markdown-content.mdx a lot and that the german translation was missing a lot of things.

Copy link

vercel bot commented Apr 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview May 2, 2024 11:22am

@github-actions github-actions bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Apr 27, 2024
@astrobot-houston
Copy link
Contributor

astrobot-houston commented Apr 27, 2024

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale File Note
de basics/layouts.mdx Localization changed, will be marked as complete.
de guides/markdown-content.mdx Localization changed, will be marked as complete.
de guides/migrate-to-astro.mdx Localization changed, will be marked as complete.
de guides/troubleshooting.mdx Localization changed, will be marked as complete.
de reference/api-reference.mdx Localization changed, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Copy link
Contributor

@Maxframe Maxframe left a comment

Choose a reason for hiding this comment

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

Uff... Wahnsinns Arbeit!

Beim Durchchecken sind mir hier und dort ein paar kleinere oder größere Ungereimtheiten aufgefallen. Falls etwas in den Vorschlägen nicht so klar sein sollte, einfach nachfragen 👍

Ich würde dir evtl. auch empfehlen, die durch das Refactoring nun kaputten Links in den anderen Seiten in das PR mit zu integrieren. Also ähnlich wie hier.

In den Check Annotations dieser PR wird dir angezeigt, welche internen Links kaputt gegangen sind, und mit welchen Slugs du das beheben kannst.

src/content/docs/de/guides/markdown-content.mdx Outdated Show resolved Hide resolved
```

<ReadMore>Lies mehr über Astros [dateibasiertes Routing](/de/guides/routing/) oder Optionen für das Erstellen [dynamischer Routen](/de/guides/routing/#dynamische-routen).</ReadMore>
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
<ReadMore>Lies mehr über Astros [dateibasiertes Routing](/de/guides/routing/) oder Optionen für das Erstellen [dynamischer Routen](/de/guides/routing/#dynamische-routen).</ReadMore>
Lies mehr über das [dateibasierte Routing](/de/guides/routing/) von Astro oder über die Möglichkeiten, [dynamische Routen](/de/guides/routing/#dynamische-routen) zu erstellen.
</ReadMore>

Dieser Satz liest sich irgendwie etwas komisch. Bin mir aber auch nicht sicher, ob meine Variante besser ist.

src/content/docs/de/guides/markdown-content.mdx Outdated Show resolved Hide resolved
src/content/docs/de/guides/markdown-content.mdx Outdated Show resolved Hide resolved
### Markdown-Layouts

Markdown-Seiten unterstützen eine spezielle Frontmatter-Eigenschaft namens `layout`, die den relativen Pfad zu einer Astro-[Layout-Komponente](/de/basics/layouts/) definiert. Diese Komponente umgibt deinen Markdown-Inhalt und stellt ein Seitengerüst und alle anderen enthaltenen Elemente der Seitenvorlage bereit.
Astro stellt [Markdown- und MDX-Seiten](/de/basics/astro-pages/#markdownmdx-pages) (in `src/pages/`) mit einer speziellen Frontmatter-Eigenschaft `layout` bereit, die einen relativen Pfad (oder [_alias_](/de/guides/aliases/)) zu einer Astro [Layout-Komponente](/de/basics/layouts/#markdown-layouts).
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
Astro stellt [Markdown- und MDX-Seiten](/de/basics/astro-pages/#markdownmdx-pages) (in `src/pages/`) mit einer speziellen Frontmatter-Eigenschaft `layout` bereit, die einen relativen Pfad (oder [_alias_](/de/guides/aliases/)) zu einer Astro [Layout-Komponente](/de/basics/layouts/#markdown-layouts).
Astro stellt [Markdown- und MDX-Seiten](/de/basics/astro-pages/#markdown-seiten) (in `src/pages/`) mit einer speziellen Frontmatter-Eigenschaft `layout` bereit, die einen relativen Pfad (oder [_Alias_](/de/guides/aliases/)) zu einer Astro [Layout-Komponente](/de/basics/layouts/#markdown-layouts).

Anscheinend gibt es im Deutschen das verlinkte Kapitel nicht. Hab jetzt mal einen funktionierenden Slug ausgewählt, welchem der englischen Version am nächsten kommt. #NWTWWHB 🤷

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Da wollte ich eh fragen, was dann der beste Weg wäre. Aufs Beste hoffen oder einfach den Slug im Link weglassen?

Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm.. wenn es in dem Moment ein Kapitel gibt, dass sich mit dem im Link angesprochen Thema befasst, würde denjenigen Slug schon verwenden.
Falls es wirklich nichts passendes geben sollte, dann würde ich persönlich auch den Slug einfach ganz weglassen und auf die Seite verweisen.

Scheint mir der beste Kompromiss zu sein

src/content/docs/de/guides/markdown-content.mdx Outdated Show resolved Hide resolved
src/content/docs/de/guides/markdown-content.mdx Outdated Show resolved Hide resolved
@@ -560,3 +655,22 @@ Wenn du Prism verwenden willst, musst du ein Stylesheet zur Syntaxhervorhebung z
4. Lade es im [`<head>`-Abschnitt deiner Seite](/de/basics/astro-pages/#seiten-html) mit einem `<link>`-Tag.
Copy link
Contributor

Choose a reason for hiding this comment

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

In Zeile 652 würde glaube ich der Slug #syntaxhervorhebung der richtige sein. GitHub lässt mich aber aus irgendeinen Grund dort keinen Vorschlag erstellen :/


## Remote-Markdown abrufen

Astro wurde ursprünglich für lokale Markdown-Dateien, die in deinem Projekt&shy;verzeichnis gespeichert sind, entwickelt. Allerdings gibt es eventuell Situationen, in denen du Markdown von einer Remote-Quelle abrufen musst.Zum Beispiel musst du möglicherweise Markdown von einer entfernten API abrufen und rendern, wenn du deine Website erstellst (oder wenn ein Benutzer eine Anfrage an deine Website stellt, wenn du [SSR](/de/guides/server-side-rendering/) verwendest).
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
Astro wurde ursprünglich für lokale Markdown-Dateien, die in deinem Projekt&shy;verzeichnis gespeichert sind, entwickelt. Allerdings gibt es eventuell Situationen, in denen du Markdown von einer Remote-Quelle abrufen musst.Zum Beispiel musst du möglicherweise Markdown von einer entfernten API abrufen und rendern, wenn du deine Website erstellst (oder wenn ein Benutzer eine Anfrage an deine Website stellt, wenn du [SSR](/de/guides/server-side-rendering/) verwendest).
Astro wurde ursprünglich für lokale Markdown-Dateien, die in deinem Projekt&shy;verzeichnis gespeichert sind, entwickelt. Allerdings gibt es eventuell Situationen, in denen du Markdown von einer Remote-Quelle abrufen musst. Zum Beispiel musst du möglicherweise Markdown von einer externen API abrufen und rendern, wenn du deine Website erstellst (oder wenn ein Benutzer eine Anfrage an deine Website stellt, wenn du [SSR](/de/guides/server-side-rendering/) verwendest).


Astro wurde ursprünglich für lokale Markdown-Dateien, die in deinem Projekt&shy;verzeichnis gespeichert sind, entwickelt. Allerdings gibt es eventuell Situationen, in denen du Markdown von einer Remote-Quelle abrufen musst.Zum Beispiel musst du möglicherweise Markdown von einer entfernten API abrufen und rendern, wenn du deine Website erstellst (oder wenn ein Benutzer eine Anfrage an deine Website stellt, wenn du [SSR](/de/guides/server-side-rendering/) verwendest).

**Astro enthält keine integrierte Unterstützung für entferntes Markdown!** Um entferntes Markdown abzurufen und in HTML zu rendern, musst du deinen eigenen Markdown-Parser aus npm installieren und konfigurieren. Dies **erbt nicht** von den in Astro konfigurierten integrierten Markdown- und MDX-Einstellungen. Stelle sicher, dass du diese Einschränkungen verstehst, bevor du dies in deinem Projekt implementierst.
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
**Astro enthält keine integrierte Unterstützung für entferntes Markdown!** Um entferntes Markdown abzurufen und in HTML zu rendern, musst du deinen eigenen Markdown-Parser aus npm installieren und konfigurieren. Dies **erbt nicht** von den in Astro konfigurierten integrierten Markdown- und MDX-Einstellungen. Stelle sicher, dass du diese Einschränkungen verstehst, bevor du dies in deinem Projekt implementierst.
**Astro enthält keine integrierte Unterstützung für externes Markdown!** Um externes Markdown abzurufen und in HTML zu rendern, musst du deinen eigenen Markdown-Parser aus npm installieren und konfigurieren. Dies **erbt nicht** von den in Astro konfigurierten integrierten Markdown- und MDX-Einstellungen. Stelle sicher, dass du diese Einschränkungen verstehst, bevor du dies in deinem Projekt implementierst.

Bin mir nicht gar nicht sicher, ob entfernt ein gängige Übersetzung für remote ist. Persönlich würde ich in dem Abschnitt eher extern nehmen oder einfach remote lassen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

passt, dann nehm ich remote! wollte es eigentlich eh nicht übersetzen, hab mich dann aber doch dafür entschieden

DerTimonius and others added 17 commits April 29, 2024 14:38
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
@Maxframe
Copy link
Contributor

Maxframe commented May 2, 2024

Auf die Schnelle finde ich gerade keine weiteren Fehler 👍
Aber bei dem Umfang dieser PR sollte auf jeden Fall noch jemand anderes drüber schauen. Da gibt es sicher noch etwas, was wir beide übersehen haben.

Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants