Skip to content

🗓️ Accessible, understandable meeting info#2438

Open
JFWooten4 wants to merge 4 commits into
stellar:mainfrom
JFWooten4:youtube-transcript-2026-05-07
Open

🗓️ Accessible, understandable meeting info#2438
JFWooten4 wants to merge 4 commits into
stellar:mainfrom
JFWooten4:youtube-transcript-2026-05-07

Conversation

@JFWooten4
Copy link
Copy Markdown
Contributor

This just continues our brief conversation from #2434 (comment) to localize relevant information into viewable elements which are also read by the screen reader. The title content suggested by AI does not show on tooltip, however, so it creates a sort of shadow data, a little bit like the i18n pages from #2410. This PR keeps it all self-explanatory with a proper title for the first time in a while, which makes sense from an SEO view since people generally search for meeting content rather than the date. Moreover, the page slug and sidebar both show this same date, so we do not lose any information or searchability.

Warning

Relies on #2361 for the controls

JFWooten4 and others added 4 commits May 11, 2026 16:48
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Copilot AI review requested due to automatic review settings May 11, 2026 20:54
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the 2026-05-07 meeting notes page to use the shared YouTube embed component and to use a descriptive, topic-focused page title (rather than the meeting date), aiming to improve accessibility/SEO and keep meeting metadata visible/understandable in the rendered page.

Changes:

  • Replace the inline YouTube <iframe> embed with the shared @site/src/components/YouTube component.
  • Update the frontmatter title from the date (2026-05-07) to a topic-based title.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

></iframe>
import YouTube from "@site/src/components/YouTube";

<YouTube ID="WHYHzDF8KJ8" />
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The current title is also not descriptive. It repeats information in the starting meta which will be read now that this page title uses the MPP descriptor suggested. This would just repeat the same info.

></iframe>
import YouTube from "@site/src/components/YouTube";

<YouTube ID="WHYHzDF8KJ8" />
Copy link
Copy Markdown
Contributor Author

@JFWooten4 JFWooten4 May 11, 2026

Choose a reason for hiding this comment

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

Oh, that's kind of interesting, can it not cross-ref in the repo? PRs show up as #plaintext

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