Restructure Learning Hub navigation and add video landing pages - #2804
Open
GeekTrainer wants to merge 4 commits into
Open
Restructure Learning Hub navigation and add video landing pages#2804GeekTrainer wants to merge 4 commits into
GeekTrainer wants to merge 4 commits into
Conversation
Regroup the Learning Hub sidebar and landing page into five intent-based sections — Fundamentals, Courses, Workshops, Video content, and Additional resources — so navigation reads as understand -> learn -> practice -> watch -> look up. - astro.config.mjs: add Courses group (nesting the existing CLI for Beginners course, with a marked insertion point for Advanced CLI and App for Beginners); rename "Copilot Workshops" -> "Workshops"; add a Video content group; merge Cookbook, Glossary, and the outbound browse links into a single Additional resources group. - learning-hub/index.md: rewrite the landing page into the same five sections so prose and sidebar stay in sync. - Add learning-hub/videos/cli-for-beginners.md: an in-hub facade page linking out to the CLI for Beginners YouTube playlist. No URLs change (labels/nesting only), so no redirects are needed. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 0fc507be-cdc6-48e3-a1d4-71f9366f3ff7
- Add embedded YouTube video landing pages for the GitHub Copilot CLI for beginners and GitHub Copilot app for beginners playlists, each listing every episode with a native player and a short description. - Wire both video pages into the Video content sidebar group. - Collapse all Learning Hub sidebar groups (and nested sub-groups) by default; the active page's group still auto-expands. - Allow YouTube embeds by adding a frame-src directive (youtube-nocookie.com, youtube.com) to the Content Security Policy. - Pin the site header (position: sticky) so toolbars no longer float over content when scrolling. - Refine landing-page copy for the Courses and Video content sections. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 0fc507be-cdc6-48e3-a1d4-71f9366f3ff7
Each video landing page now links to install its respective harness — the CLI page to the Copilot CLI install docs, the app page to the GitHub Copilot app landing page — so viewers can set up before watching. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 0fc507be-cdc6-48e3-a1d4-71f9366f3ff7
github-actions
Bot
added
new-submission
Contributor
There was a problem hiding this comment.
Pull request overview
Restructures Learning Hub navigation around learning intent and introduces embedded video-series pages.
Changes:
- Reorganizes sidebar and landing-page sections.
- Adds Copilot CLI and Copilot app video pages.
- Enables responsive YouTube embeds and sticky header behavior.
Show a summary per file
| File | Description |
|---|---|
website/astro.config.mjs |
Defines the reorganized sidebar. |
website/src/components/Head.astro |
Permits YouTube frames in CSP. |
website/src/content/docs/learning-hub/index.md |
Mirrors the new navigation structure. |
website/src/content/docs/learning-hub/videos/cli-for-beginners.md |
Adds the CLI video series. |
website/src/content/docs/learning-hub/videos/copilot-app.md |
Adds the Copilot app video series. |
website/src/styles/global.css |
Styles responsive video embeds. |
website/src/styles/starlight-overrides.css |
Makes the header sticky. |
Review details
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 7/7 changed files
- Comments generated: 3
- Review effort level: Balanced
- Promote the six episode headings on each video page from h3 to h2 so the heading hierarchy doesn't skip a level after the h1 title (fixes a screen-reader navigation issue). - Link both the CLI and app video series from the landing page's Video content section, so neither destination is sidebar-only. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 0fc507be-cdc6-48e3-a1d4-71f9366f3ff7
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Restructures the Learning Hub so its navigation reads by intent — understand → learn → practice → watch → look up — and adds video landing pages. Today the sidebar is a flat pile of top-level groups mixing concepts, a course, a workshop, a cookbook, and outbound links, which doesn't scale as new content lands.
Sidebar → five intent-based groups
All groups (and nested sub-groups) are collapsed by default; the active page's group still auto-expands. No URLs change — only labels and nesting — so no redirects are needed.
Video landing pages
learning-hub/videos/cli-for-beginners— GitHub Copilot CLI for beginnerslearning-hub/videos/copilot-app— GitHub Copilot app for beginnersEach lists every episode with a natively embedded, playable YouTube player (privacy-friendly
youtube-nocookie.com, lazy-loaded), a short description, and an install link for the respective harness.Supporting changes
index.md) to mirror the five sections.frame-srcdirective (youtube-nocookie.com,youtube.com) to the Content Security Policy inHead.astroso the embeds are allowed.position: stickyso toolbars no longer float over content while scrolling..video-embedresponsive-embed rule toglobal.css.Validation
npm run website:data+npm run buildpass; the new video pages and localized fallbacks build with no unresolved sidebar slugs.Related
This PR is intended to land first; PR #2800 (Advanced Copilot CLI) will stack on top of it, nesting its course under the new Courses group.