Skip to content

docs: fix accessibility and build issues from multi-device testing report #21912

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.

Sign up for GitHub

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

Closed
Copilot wants to merge 2 commits into from
Closed

docs: fix accessibility and build issues from multi-device testing report #21912

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d8842c1
Initial plan
Copilot Mar 20, 2026
69f7453
docs: fix accessibility and docs issues from testing report
Copilot Mar 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion docs/astro.config.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -165,12 +165,13 @@ export default defineConfig({
"markdown",
"yaml"
]),
langAlias: { aw: "markdown" }
langAlias: { aw: "markdown", abnf: "text" }
},
},
plugins: [
starlightBlog({
recentPostCount: 12,
navigation: 'none',
authors: createAuthors({
'githubnext': {
name: 'GitHub Next',
Expand Down
14 changes: 12 additions & 2 deletions docs/src/components/CustomHead.astro
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,18 +29,28 @@ const overriddenProps = new Set([
'twitter:title', 'twitter:description', 'twitter:image',
]);

// Filter Starlight's head tags to remove OG/Twitter tags we'll override
// Generate the correct page title (avoids "Site | Site" duplicate on homepage)
const pageTitle = isHomePage ? siteTitle : `${rawPageTitle} | ${siteTitle}`;

Comment on lines +32 to +34
Copy link

Copilot AI Mar 20, 2026

Choose a reason for hiding this comment

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

pageTitle duplicates the same conditional formatting already used for ogTitle. Consider deriving ogTitle (and the Twitter title) from pageTitle to keep the HTML <title> and OG/Twitter titles from drifting if the formatting logic changes later.

Copilot uses AI. Check for mistakes.
// Filter Starlight's head tags to remove OG/Twitter tags we'll override.
// Also filter the <title> tag: Starlight generates "Page Title | Site Title" for every page,
// which produces "GitHub Agentic Workflows | GitHub Agentic Workflows" on the homepage
// because the page title and site title are identical. We output our own deduplicated title below.
const head = route?.head ?? [];
const filteredHead = head.filter(({ tag, attrs }) => {
if (tag === 'title') return false;
if (tag !== 'meta') return true;
const prop = attrs?.property || attrs?.name;
return !overriddenProps.has(prop);
});
---

<!-- Starlight's head tags (with OG/Twitter duplicates removed) -->
<!-- Starlight's head tags (with OG/Twitter duplicates and title removed) -->
{filteredHead.map(({ tag: Tag, attrs, content }) => <Tag {...attrs} set:html={content} />)}

<!-- Page title (deduplicated: avoids "GitHub Agentic Workflows | GitHub Agentic Workflows" on homepage) -->
<title>{pageTitle}</title>

<!-- Enhanced OG tags with site name in title and social image -->
<meta property="og:title" content={ogTitle} />
<meta property="og:description" content={pageDescription} />
Expand Down
2 changes: 1 addition & 1 deletion docs/src/components/CustomLogo.astro
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ const initialLogoUrl = darkLogoUrl;
---

<a href={homeUrl} class="site-title">
<img src={initialLogoUrl} alt="GitHub Agentic Workflows" class="theme-logo" data-dark-src={darkLogoUrl} data-light-src={lightLogoUrl} />
<img src={initialLogoUrl} alt="" class="theme-logo" data-dark-src={darkLogoUrl} data-light-src={lightLogoUrl} />
<span>GitHub Agentic Workflows</span>
</a>

Expand Down
5 changes: 5 additions & 0 deletions docs/src/components/ThemeToggle.astro
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,16 @@
---
import octicons from '@primer/octicons';
import BlogThemeSelect from 'starlight-blog/components/ThemeSelect.astro';

const sunIcon = octicons.sun.toSVG({ width: 16, height: 16 });
const moonIcon = octicons.moon.toSVG({ width: 16, height: 16 });
const autoIcon = octicons['device-desktop'].toSVG({ width: 16, height: 16 });
---

<!-- Blog navigation link: starlight-blog's "ThemeSelect" override adds a "Blog" header link
(not a theme control). We include it here to delegate properly since astro.config.mjs sets
navigation: 'none' to suppress the plugin's duplicate-override warning. -->
<BlogThemeSelect />
<starlight-theme-select>
<label class="theme-label">
<span class="sr-only">Select theme</span>
Expand Down
2 changes: 2 additions & 0 deletions docs/src/content/docs/agent-factory-status.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,7 @@ These are experimental agentic workflows used by the GitHub Next team to learn,
| [Daily Compiler Quality Check](https://github.com/github/gh-aw/blob/main/.github/workflows/daily-compiler-quality.md) | copilot | [![Daily Compiler Quality Check](https://github.com/github/gh-aw/actions/workflows/daily-compiler-quality.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/daily-compiler-quality.lock.yml) | - | - |
| [Daily Copilot PR Merged Report](https://github.com/github/gh-aw/blob/main/.github/workflows/copilot-pr-merged-report.md) | copilot | [![Daily Copilot PR Merged Report](https://github.com/github/gh-aw/actions/workflows/copilot-pr-merged-report.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/copilot-pr-merged-report.lock.yml) | `0 15 * * 1-5` | - |
| [Daily Copilot Token Consumption Report](https://github.com/github/gh-aw/blob/main/.github/workflows/daily-copilot-token-report.md) | copilot | [![Daily Copilot Token Consumption Report](https://github.com/github/gh-aw/actions/workflows/daily-copilot-token-report.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/daily-copilot-token-report.lock.yml) | `0 11 * * 1-5` | - |
| [Daily DIFC Integrity-Filtered Events Analyzer](https://github.com/github/gh-aw/blob/main/.github/workflows/daily-integrity-analysis.md) | copilot | [![Daily DIFC Integrity-Filtered Events Analyzer](https://github.com/github/gh-aw/actions/workflows/daily-integrity-analysis.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/daily-integrity-analysis.lock.yml) | - | - |
| [Daily Documentation Healer](https://github.com/github/gh-aw/blob/main/.github/workflows/daily-doc-healer.md) | claude | [![Daily Documentation Healer](https://github.com/github/gh-aw/actions/workflows/daily-doc-healer.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/daily-doc-healer.lock.yml) | - | - |
| [Daily Documentation Updater](https://github.com/github/gh-aw/blob/main/.github/workflows/daily-doc-updater.md) | claude | [![Daily Documentation Updater](https://github.com/github/gh-aw/actions/workflows/daily-doc-updater.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/daily-doc-updater.lock.yml) | - | - |
| [Daily Fact About gh-aw](https://github.com/github/gh-aw/blob/main/.github/workflows/daily-fact.md) | codex | [![Daily Fact About gh-aw](https://github.com/github/gh-aw/actions/workflows/daily-fact.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/daily-fact.lock.yml) | `0 11 * * 1-5` | - |
Expand Down Expand Up @@ -132,6 +133,7 @@ These are experimental agentic workflows used by the GitHub Next team to learn,
| [Resource Summarizer Agent](https://github.com/github/gh-aw/blob/main/.github/workflows/pdf-summary.md) | copilot | [![Resource Summarizer Agent](https://github.com/github/gh-aw/actions/workflows/pdf-summary.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/pdf-summary.lock.yml) | - | `/summarize` |
| [Safe Output Health Monitor](https://github.com/github/gh-aw/blob/main/.github/workflows/safe-output-health.md) | claude | [![Safe Output Health Monitor](https://github.com/github/gh-aw/actions/workflows/safe-output-health.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/safe-output-health.lock.yml) | - | - |
| [Schema Consistency Checker](https://github.com/github/gh-aw/blob/main/.github/workflows/schema-consistency-checker.md) | claude | [![Schema Consistency Checker](https://github.com/github/gh-aw/actions/workflows/schema-consistency-checker.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/schema-consistency-checker.lock.yml) | - | - |
| [Schema Feature Coverage Checker](https://github.com/github/gh-aw/blob/main/.github/workflows/schema-feature-coverage.md) | codex | [![Schema Feature Coverage Checker](https://github.com/github/gh-aw/actions/workflows/schema-feature-coverage.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/schema-feature-coverage.lock.yml) | - | - |
| [Scout](https://github.com/github/gh-aw/blob/main/.github/workflows/scout.md) | claude | [![Scout](https://github.com/github/gh-aw/actions/workflows/scout.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/scout.lock.yml) | - | `/scout` |
| [Security Alert Burndown](https://github.com/github/gh-aw/blob/main/.github/workflows/security-alert-burndown.campaign.g.md) | claude | [![Security Alert Burndown](https://github.com/github/gh-aw/actions/workflows/security-alert-burndown.campaign.g.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/security-alert-burndown.campaign.g.lock.yml) | `0 18 * * *` | - |
| [Security Compliance Campaign](https://github.com/github/gh-aw/blob/main/.github/workflows/security-compliance.md) | copilot | [![Security Compliance Campaign](https://github.com/github/gh-aw/actions/workflows/security-compliance.lock.yml/badge.svg)](https://github.com/github/gh-aw/actions/workflows/security-compliance.lock.yml) | - | - |
Expand Down
2 changes: 0 additions & 2 deletions docs/src/content/docs/reference/fuzzy-schedule-specification.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@ sidebar:
order: 1360
---

# Fuzzy Schedule Time Syntax Specification

**Version**: 1.1.0
**Status**: Draft Specification
**Latest Version**: [fuzzy-schedule-specification](/gh-aw/reference/fuzzy-schedule-specification/)
Expand Down
13 changes: 12 additions & 1 deletion docs/src/scripts/responsive-tables.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,21 @@ function enhanceResponsiveTables() {
tables.forEach(table => {
const headers: string[] = [];

// Extract header text from thead
// Extract header text from thead and add scope="col" for accessibility
const headerCells = table.querySelectorAll('thead th');
headerCells.forEach(th => {
headers.push(th.textContent?.trim() || '');
if (!th.hasAttribute('scope')) {
th.setAttribute('scope', 'col');
}
});

// Add scope="row" to row headers in tbody
const bodyHeaderCells = table.querySelectorAll('tbody th');
bodyHeaderCells.forEach(th => {
if (!th.hasAttribute('scope')) {
th.setAttribute('scope', 'row');
}
});

// Add data-label attribute to each td based on column position
Expand Down
Loading