Skip to content

Conversation

@dmols dmols self-requested a review October 2, 2025 17:24
Copy link
Contributor

@dmols dmols left a comment

Choose a reason for hiding this comment

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

Not particularly sure how to test the <canvas> and <input> specific condition changes for the AltText form, seeing as both are not allowed in Canvas. Could do so in an html file but since the file replacement feature is not fleshed out (iirc), it may be difficult to test if it works. Open to suggestions!

That said, I did test the rest of the issues fixed and I saw no errors pop up except for the "Heading Empty" and "Links Should be Descriptive" fix. This is what I saw when I tried to remove a heading.

Screen.Recording.2025-10-07.at.9.55.01.AM.mov

Same thing occurs for the links, though sometimes clicking "Show Content Preview" just loads a page with the word 'undefined' in it, like so:

Screenshot 2025-10-07 at 9 59 07 AM

Worth noting, I did clear my code cache and browser cache, and still encountered the same issue.

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