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

Deconstruct library tour #5367

Merged
merged 23 commits into from
Dec 18, 2023
Merged

Deconstruct library tour #5367

merged 23 commits into from
Dec 18, 2023

Conversation

MaryaBelanger
Copy link
Contributor

@MaryaBelanger MaryaBelanger commented Nov 21, 2023

This PR breaks up the library tour to facilitate more content living under "Core libraries" in the future and users being able to access what is important to them more easily.

The change is implemented in a similar fashion to the language tour split up. The following work was completed:

  • Introduced a new top-level /libraries folder
  • Combined the index at /guides/libraries and the intro to the library tour at /libraries/index.
  • Split each section of the library tour in to their own page
  • Add each new page to the sidenav under "Core libraries"
  • Add a redirector on the old page to redirect accesses to specific headings to the correct new page and heading
  • Add previous and next buttons to the various pages
  • Update links across the entire website
  • Add and update redirects where necessary

Fixes #4610

@MaryaBelanger MaryaBelanger self-assigned this Nov 21, 2023
@parlough parlough assigned parlough and unassigned MaryaBelanger Dec 14, 2023
Copy link

github-actions bot commented Dec 14, 2023

Visit the preview URL for this PR (updated for commit f1b5ed3):

https://dart-dev--pr5367-library-tour-mqpt3qxg.web.app

(expires Mon, 25 Dec 2023 23:13:53 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: d851bc446d3c4d7394c5406c6f07255afc7075f3

@parlough parlough marked this pull request as ready for review December 18, 2023 20:01
@MaryaBelanger
Copy link
Contributor Author

Thanks for taking this over and doing all the heavy lifting with the redirects @parlough! I picked a handful of links around the site to test and everything looks good. The layout makes sense to me too. If you want to move https://dart.dev/codelabs/async-await and https://dart.dev/tutorials/language/streams up to your new "library guides" section, you can go ahead and do that. Either way, consider this "approved" by me!

Copy link
Member

@parlough parlough left a comment

Choose a reason for hiding this comment

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

Deferring to @MaryaBelanger's review in #5367 (comment) since I took over ownership of this PR.

Thanks for starting this and reviewing it! I moved the using streams article up too :)

@parlough parlough merged commit 14ef584 into main Dec 18, 2023
9 checks passed
@parlough parlough deleted the library-tour branch December 18, 2023 23:23
atsansone pushed a commit to atsansone/site-www that referenced this pull request Jan 26, 2024
This PR breaks up the library tour to facilitate more content living
under "Core libraries" in the future and users being able to access what
is important to them more easily.

The change is implemented in a similar fashion to the language tour
split up. The following work was completed:

- Introduced a new top-level `/libraries` folder
- Combined the index at `/guides/libraries` and the intro to the library
tour at `/libraries/index`.
- Split each section of the library tour in to their own page
- Add each new page to the sidenav under "Core libraries"
- Add a redirector on the old page to redirect accesses to specific
headings to the correct new page and heading
- Add previous and next buttons to the various pages
- Update links across the entire website
- Add and update redirects where necessary

Fixes dart-lang#4610

---------

Co-authored-by: Parker Lougheed <[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.

☂ [Tracking] Refactor "Core libraries" umbrella topic
2 participants