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

Tweaks to improve “link” search results #2091

Merged
merged 4 commits into from
Nov 22, 2022
Merged

Conversation

delucis
Copy link
Member

@delucis delucis commented Nov 22, 2022

What kind of changes does this PR include?

  • Minor content fixes (broken links, typos, etc.)

Description

  • Remove “link” from page titles in tutorial lessons
  • Add a “Link between pages” subheading to our “Pages” page to help that content stand out

Probably not a silver bullet because “Link between pages” is <h3> and there are some <h2>s containing “link” in the tutorial, but should be a start. We can re-evaluate if we want to do more after re-crawling.

@netlify
Copy link

netlify bot commented Nov 22, 2022

Deploy Preview for astro-docs-2 ready!

Name Link
🔨 Latest commit 73919fa
🔍 Latest deploy log https://app.netlify.com/sites/astro-docs-2/deploys/637ce3e7cdd2ab0009655523
😎 Deploy Preview https://deploy-preview-2091--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@delucis delucis self-assigned this Nov 22, 2022
Co-authored-by: Sarah Rainsberger <[email protected]>
@delucis delucis merged commit f3193c3 into main Nov 22, 2022
@delucis delucis deleted the chris/link-results branch November 22, 2022 15:22
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 <[email protected]>

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

Co-authored-by: Max <[email protected]>

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

Co-authored-by: Max <[email protected]>

* revert Update markdown-content.mdx

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

---------

Co-authored-by: Max <[email protected]>
Co-authored-by: Yan <[email protected]>
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.

2 participants