Skip to content

🗓️ Accessible, understandable meeting info#2438

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

🗓️ 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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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" />

@JFWooten4 JFWooten4 May 11, 2026

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.

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

@ElliotFriend

Copy link
Copy Markdown
Contributor

Thanks — closing this one because events overtook it: #2361 merged today and already converted this file's iframe to the shared <YouTube> component, which was the substantive improvement here.

What remains in this diff is retitling the note from "2026-05-07" to "Machine Payments Protocol Deep Dive". I'm declining that for now: every meeting note currently uses the date-title convention, and a one-off exception makes the series inconsistent. If we decide topic-based titles are better (there's a real argument!), we should change the convention repo-wide in one pass — feel free to open an issue making that case.

@JFWooten4
JFWooten4 deleted the youtube-transcript-2026-05-07 branch July 14, 2026 11:20
@JFWooten4
JFWooten4 restored the youtube-transcript-2026-05-07 branch July 19, 2026 10:55
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.

3 participants