Skip to content

Conversation

@Tristan-Raz
Copy link
Contributor

Fixes issue #256

@pearcej
Copy link
Owner

pearcej commented May 15, 2025

I don't think this follows the PreTeXt standards. Please see https://pretextbook.org/doc/guide/html/overview-images.html#overview-images-9.

@Tristan-Raz
Copy link
Contributor Author

My mistake, I will correct it!

@Tristan-Raz Tristan-Raz marked this pull request as ready for review May 19, 2025 13:49
@Tristan-Raz
Copy link
Contributor Author

Signed off on by Colin and Elijah.

@pearcej
Copy link
Owner

pearcej commented May 20, 2025

@Tristan-Raz This is still not correct. Please use <description> and <shortdescription>.

pearcej
pearcej previously approved these changes May 20, 2025
Copy link
Owner

@pearcej pearcej left a comment

Choose a reason for hiding this comment

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

@Tristan-Raz You missed a couple of the ones with the incorrect <alt> tag.

@pearcej pearcej dismissed their stale review May 20, 2025 01:02

Hit wrong button

Copy link
Owner

@pearcej pearcej left a comment

Choose a reason for hiding this comment

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

@Tristan-Raz You missed a couple of the ones with the incorrect <alt> tag. Can you please fix?

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