Skip to content

Conversation

giacomo-petri
Copy link
Collaborator

Closes: #2241

This PR is intended to address how browsers are differently exposing SVG elements.

Changes:

Need for Call for Review:
This will require a 2 weeks Call for Review

@giacomo-petri
Copy link
Collaborator Author

giacomo-petri commented Feb 7, 2025

I've just realised there is one more rule affected by this. I'm adding the file to have everything in one go

Copy link
Collaborator

@iadawn iadawn left a comment

Choose a reason for hiding this comment

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

Minor suggestion to make finding the reference a bit easier

@giacomo-petri
Copy link
Collaborator Author

We can move forward with the review, since the related issue requires necessarily another issue and PR.

Copy link

netlify bot commented Feb 13, 2025

Deploy Preview for act-rules ready!

Name Link
🔨 Latest commit cadc07d
🔍 Latest deploy log https://app.netlify.com/sites/act-rules/deploys/67d2e19e333d800008143c3f
😎 Deploy Preview https://deploy-preview-2290--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 site configuration.

@giacomo-petri
Copy link
Collaborator Author

Call for review ends on February 28th 2025.

@giacomo-petri giacomo-petri added Review call 2 weeks Call for review for new rules and big changes and removed reviewers wanted labels Feb 13, 2025
@daniel-montalvo daniel-montalvo force-pushed the giacomo-petri-patch-svg-acc-supp-note-23a2a8 branch from 352e7a8 to cadc07d Compare March 13, 2025 13:46
@carlosapaduarte carlosapaduarte merged commit 4a3a75d into develop Mar 17, 2025
6 checks passed
@carlosapaduarte carlosapaduarte deleted the giacomo-petri-patch-svg-acc-supp-note-23a2a8 branch March 17, 2025 14:18
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[23a2a8] Image has non-empty accessible name - Inapplicable Example 1 is arguable

7 participants