Skip to content

Show fig-alt in the taught snippets of two examples - #2143

Draft
cwickham wants to merge 1 commit into
mainfrom
fix/show-fig-alt-in-taught-snippets
Draft

Show fig-alt in the taught snippets of two examples#2143
cwickham wants to merge 1 commit into
mainfrom
fix/show-fig-alt-in-taught-snippets

Conversation

@cwickham

Copy link
Copy Markdown
Member

Follow-up to #2138, which added fig-alt to the live copies of two examples but not to the shown syntax next to them. A reader who inspects the page HTML would see an alt attribute that the shown syntax does not produce.

This PR adds the same fig-alt to the taught snippets, with placeholder values that match the neighboring options:

  • _cross-references-listings.qmd: add #| fig-alt: Figure alt text to the shown code cell, next to fig-cap: Figure caption.
  • cross-references-divs.qmd: add {fig-alt="Alt text"} to the shown ![](table.png) in the div syntax example.

The freeze for cross-references.qmd (which includes the listings partial) is re-executed. cross-references-divs.qmd uses the markdown engine and has no freeze entry.

PR #2138 added fig-alt to the live copies of two examples but not to
the shown syntax next to them. A reader who inspects the page HTML
would see an alt attribute that the shown syntax does not produce.

Add the same fig-alt to the taught snippets, with placeholder values
that match the neighboring options:

- _cross-references-listings.qmd: add `#| fig-alt: Figure alt text`
  to the shown code cell, next to `fig-cap: Figure caption`.
- cross-references-divs.qmd: add `{fig-alt="Alt text"}` to the shown
  `![](table.png)` in the div syntax example.

Re-execute the freeze for cross-references.qmd, which includes the
listings partial. cross-references-divs.qmd uses the markdown engine
and has no freeze entry.
@github-actions

Copy link
Copy Markdown
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-2143.quarto.org

🔄 Modified Documents

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