Skip to content

fix: add spacing to 'added in' texts#5765

Merged
delucis merged 3 commits intomainfrom
add-p
Dec 13, 2023
Merged

fix: add spacing to 'added in' texts#5765
delucis merged 3 commits intomainfrom
add-p

Conversation

@kevinzunigacuellar
Copy link
Copy Markdown
Member

Description

Add spacing to <Since /> elements

Before After
image image

@vercel
Copy link
Copy Markdown

vercel bot commented Dec 13, 2023

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

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview Dec 13, 2023 6:14pm

@ElianCodes
Copy link
Copy Markdown
Member

ElianCodes commented Dec 13, 2023

Wouldn't it make more sense to add padding to the <Since /> component instead of wrapping all of them in a <p />?

@ElianCodes ElianCodes self-requested a review December 13, 2023 08:53
@kevinzunigacuellar
Copy link
Copy Markdown
Member Author

Wouldn't it make more sense to add padding to the <Since /> component instead of wrapping all of them in a <p />?

It would but there are some instances of Since that are inlined 😅

@ElianCodes
Copy link
Copy Markdown
Member

in that case, what about a prop for spacing, that defaults to none?

Copy link
Copy Markdown
Member

@delucis delucis left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for tackling this @kevinzunigacuellar 💜

@sarah11918 sarah11918 added site improvement Some thing that improves the website functionality - ask @delucis for help! Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)! labels Dec 13, 2023
@sarah11918
Copy link
Copy Markdown
Member

Thanks, Kevin! I updated Astro Docs Docs to show this usage of the component in a standalone line!

dreyfus92 added a commit that referenced this pull request Jan 14, 2024
…mdx` (#6363)

* i18n(fr) Update reading-time.mdx

Just for translator tracker because the PR #5766 should have been [ignore]

* Update astro-pages.mdx

* Update framework-components.mdx

* Update layouts.mdx

* Update routing.mdx

* Update editor-setup.mdx

* Update configuring-astro.mdx

* Update cloudflare.mdx

* Update deno.mdx

* Update netlify.mdx

* Update vercel.mdx

* Update imports.mdx

* Update auto.mdx

* Update manual.mdx

* Update cli-reference.mdx

Update and del some parts

* Update cli-reference.mdx

Update fiule with PR #5512 #5651 #5604 #5765 #6040 #6043 #6109 #6267

* Update auto.mdx

* Update cloudflare.mdx

fix PR #5284

* Update markdown-content.mdx

* Update auto.mdx

* Update src/content/docs/fr/core-concepts/astro-pages.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/core-concepts/astro-pages.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/core-concepts/astro-pages.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/core-concepts/framework-components.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/core-concepts/framework-components.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/guides/markdown-content.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/install/manual.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/install/manual.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/reference/cli-reference.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Apply suggestions from code review

---------

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>
ematipico pushed a commit that referenced this pull request Jan 26, 2024
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Chris Swithinbank <swithinbank@gmail.com>
Co-authored-by: Elian <15145918+ElianCodes@users.noreply.github.com>
ematipico pushed a commit that referenced this pull request Jan 26, 2024
Update french translation for rss.mdx file with PR #5765

Co-authored-by: Yan Thomas <61414485+Yan-Thomas@users.noreply.github.com>
ematipico pushed a commit that referenced this pull request Jan 26, 2024
…mdx` (#6363)

* i18n(fr) Update reading-time.mdx

Just for translator tracker because the PR #5766 should have been [ignore]

* Update astro-pages.mdx

* Update framework-components.mdx

* Update layouts.mdx

* Update routing.mdx

* Update editor-setup.mdx

* Update configuring-astro.mdx

* Update cloudflare.mdx

* Update deno.mdx

* Update netlify.mdx

* Update vercel.mdx

* Update imports.mdx

* Update auto.mdx

* Update manual.mdx

* Update cli-reference.mdx

Update and del some parts

* Update cli-reference.mdx

Update fiule with PR #5512 #5651 #5604 #5765 #6040 #6043 #6109 #6267

* Update auto.mdx

* Update cloudflare.mdx

fix PR #5284

* Update markdown-content.mdx

* Update auto.mdx

* Update src/content/docs/fr/core-concepts/astro-pages.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/core-concepts/astro-pages.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/core-concepts/astro-pages.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/core-concepts/framework-components.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/core-concepts/framework-components.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/guides/markdown-content.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/install/manual.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/install/manual.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Update src/content/docs/fr/reference/cli-reference.mdx

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>

* Apply suggestions from code review

---------

Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>
fhiromasa added a commit to fhiromasa/astro-docs that referenced this pull request Mar 3, 2024
fix: add spacing to 'added in' texts (withastro#5765)
yanthomasdev added a commit that referenced this pull request Mar 5, 2024
* Update routing.mdx in ja

Modified to be similar to the English sample code in "Nested Pagination" section

* update routing.mdx in ja

[code-example-syntax] hippo-proof code examples (#5364)

* update routing.mdx in ja

fix: add spacing to 'added in' texts (#5765)

* update routing.mdx in ja

Add <ReadMore> component (#6267)

* Update routing.mdx in ja

Document new routing priority behavior (#6386)

* Update src/content/docs/ja/guides/routing.mdx

Co-authored-by: Shinya Fujino <shf0811@gmail.com>

* Update src/content/docs/ja/guides/routing.mdx

Co-authored-by: Shinya Fujino <shf0811@gmail.com>

* Update src/content/docs/ja/guides/routing.mdx

Co-authored-by: Shinya Fujino <shf0811@gmail.com>

* Update src/content/docs/ja/guides/routing.mdx

Co-authored-by: Shinya Fujino <shf0811@gmail.com>

* Update src/content/docs/ja/guides/routing.mdx

Co-authored-by: Shinya Fujino <shf0811@gmail.com>

* Update src/content/docs/ja/guides/routing.mdx

Co-authored-by: Shinya Fujino <shf0811@gmail.com>

---------

Co-authored-by: Shinya Fujino <shf0811@gmail.com>
Co-authored-by: Yan <61414485+yanthomasdev@users.noreply.github.com>
trueberryless added a commit to trueberryless/withastro-docs that referenced this pull request Oct 22, 2024
yanthomasdev added a commit that referenced this pull request Nov 4, 2024
* update translation #1442

* update translation #1434

* update translation #1510

* update translation #1538

* Update src/content/docs/de/basics/astro-pages.mdx

* Update src/content/docs/de/basics/astro-pages.mdx

* Update src/content/docs/de/basics/astro-pages.mdx

* Update src/content/docs/de/basics/astro-pages.mdx

* Update src/content/docs/de/basics/astro-pages.mdx

* update translation #1638

* update translation #2091

* update translation #2133

* update translation #2409

* update translation #2371

* update translation #4610

* update translation #5128

* update translation #5205

* update translation #5240

* update translation #5364

* update translation #5765

* update translation #6267

* remove paragraph where I cant find when it was deleted in original version

* update translation #6620

* update translation #8495

* update translation #8573

* update translation #9336

* update translation #9336 2/2

didnt save file locally, so git didnt add

* fix all visual differences by comparing manually against original english version

* fix broken links in `astro-pages.mdx`

* Breaking changes to other files! fixing links which link to the �stro-pages.mdx file

* but now...

* Update src/content/docs/de/basics/astro-pages.mdx

Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>

* Update src/content/docs/de/basics/astro-pages.mdx

Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>

* Update src/content/docs/de/basics/astro-pages.mdx

Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>

* revert Update markdown-content.mdx

@lunaria-track:src/content/docs/de/basics/astro-pages.mdx

---------

Co-authored-by: Max <51922004+Maxframe@users.noreply.github.com>
Co-authored-by: Yan <61414485+yanthomasdev@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)! site improvement Some thing that improves the website functionality - ask @delucis for help!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants