Skip to content

Moved Failed Example 4 into Passed Example 9 in HTML images contain no text [0va7u6]#2398

Open
giacomo-petri wants to merge 5 commits into
developfrom
giacomo-petri-0va7u6-failed-4
Open

Moved Failed Example 4 into Passed Example 9 in HTML images contain no text [0va7u6]#2398
giacomo-petri wants to merge 5 commits into
developfrom
giacomo-petri-0va7u6-failed-4

Conversation

@giacomo-petri

Copy link
Copy Markdown
Collaborator

Closes issue(s): #2397

This PR moves failed example 4 to passing example 9, as SC 1.4.5 Images of Text clearly states in its note that when the same information conveyed by an image of text is also available as real text, the requirement is satisfied.

Need for Call for Review:
This will require a 1 week Call for Review


How to Review And Approve

  • Go to the “Files changed” tab
  • Here you will have the option to leave comments on different lines.
  • Once the review is completed, find the “Review changes” button in the top right, select “Approve” (if you are really confident in the rule) or "Request changes" and click “Submit review”.
  • Make sure to also review the proposed Call for Review period. In case of disagreement, the longer period wins.

@giacomo-petri giacomo-petri requested a review from WilcoFiers April 2, 2026 13:35
@giacomo-petri giacomo-petri self-assigned this Apr 2, 2026
@giacomo-petri giacomo-petri added the Rule Update Use this label for an existing rule that is being updated label Apr 2, 2026
@netlify

netlify Bot commented Apr 2, 2026

Copy link
Copy Markdown

Deploy Preview for act-rules ready!

Name Link
🔨 Latest commit 1e57574
🔍 Latest deploy log https://app.netlify.com/projects/act-rules/deploys/69e09feedfde590008f70644
😎 Deploy Preview https://deploy-preview-2398--act-rules.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.

Comment thread _rules/image-no-text-0va7u6.md Outdated
@giacomo-petri

giacomo-petri commented Apr 16, 2026

Copy link
Copy Markdown
Collaborator Author

I need to make an adjustment regarding 1.4.9 since it's mapped as well. I'm gonna mark 1.4.9 as secondary requirement (this success criterion is more strict than this rule).

@Jym77 Jym77 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Background note should link to the Understanding document since it is where the note is.

Comment thread _rules/image-no-text-0va7u6.md Outdated
@WilcoFiers WilcoFiers added the Review Call 1 week Call for review for small changes label May 4, 2026
@giacomo-petri

Copy link
Copy Markdown
Collaborator Author

Call for review ends May 12th 2026

@giacomo-petri

Copy link
Copy Markdown
Collaborator Author

As @WilcoFiers suggested, since there are days off this week and next we are extending the review period to two weeks. Call for reviews ends May 19th 2026

@giacomo-petri giacomo-petri added Review call 2 weeks Call for review for new rules and big changes and removed Review Call 1 week Call for review for small changes labels May 4, 2026

@sorinfratila sorinfratila left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

does this need to have the 'updated at' date updated ? I noticed there is this kind of info on the act rule pages.

@Jym77

Jym77 commented May 28, 2026

Copy link
Copy Markdown
Collaborator

does this need to have the 'updated at' date updated ? I noticed there is this kind of info on the act rule pages.

No, there is some magic happening to do that for us (looking at git logs).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review call 2 weeks Call for review for new rules and big changes reviewers wanted Rule Update Use this label for an existing rule that is being updated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants