Skip to content

docs(slides): legend wrap and alt text for GHE-repos figure#4

Open
larnsce wants to merge 3 commits into
mainfrom
dev
Open

docs(slides): legend wrap and alt text for GHE-repos figure#4
larnsce wants to merge 3 commits into
mainfrom
dev

Conversation

@larnsce

@larnsce larnsce commented Jun 1, 2026

Copy link
Copy Markdown
Member

Summary

  • Switch the legend in the GHE-repos figure from nrow = 1 to nrow = 2 so all 9 license categories are visible. Previously Apache-2.0 and CC-BY-SA-4.0 were clipped.
  • Add a descriptive fig-alt to the chunk so the rendered slide carries a meaningful image description (license-by-year breakdown plus the headline 28-of-74 no-license stat) for screen-reader users.
  • Includes a local site render commit (15c215b) with refreshed _freeze and slides_files artifacts that pick up the new figure.

Test plan

  • Rscript slides/fig-ghe-github-repos-by-license.R runs cleanly and writes slides/img/fig-ghe-github-repos-by-license.png at 1200x627; all 9 license categories visible in the legend.
  • slides/reproducibilitea.qmd renders without errors and the inline figure on the ghedata slide shows the same legend wrap.
  • The rendered HTML's GHE-repos figure carries a non-empty alt attribute matching the fig-alt text.
  • Visual check (human eyes): figure looks reasonable; legend wraps cleanly and does not overflow.

larnsce and others added 2 commits June 1, 2026 09:55
Previously the legend was forced to one row with `nrow = 1`, which
clipped two of the nine license categories (Apache-2.0, CC-BY-SA-4.0)
out of view. Switch to `nrow = 2` so every license is visible. Re-render
the standalone PNG accordingly.

Add a descriptive fig-alt to the chunk so the rendered slide carries
a meaningful image description (license-by-year breakdown and the
headline 28-of-74 no-license stat) for screen-reader users.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@netlify

netlify Bot commented Jun 1, 2026

Copy link
Copy Markdown

Deploy Preview for idyllic-mousse-66a099 ready!

Name Link
🔨 Latest commit cc4d17b
🔍 Latest deploy log https://app.netlify.com/projects/idyllic-mousse-66a099/deploys/6a1eb38d7be1170008199635
😎 Deploy Preview https://deploy-preview-4--idyllic-mousse-66a099.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant