-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add aria-describedby if image has no alt but has caption #43059
Closed
Closed
Changes from 54 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
fb4e0b3
Add aria-describedby if image has no alt but has caption
carolinan fdb01d0
Try to update fixtures
carolinan 889ed6c
Update blocks-raw-handling.test.js.snap
carolinan 6e504a8
try to update native test files
carolinan 77c73c4
revert native test change
carolinan c3bf8fa
Merge branch 'trunk' of github.com:wordpress/gutenberg into update/im…
alexstine ad3be98
Update snapshot.
alexstine 15e7b5e
Merge branch 'trunk' of github.com:wordpress/gutenberg into update/im…
alexstine 3792981
Merge branch 'trunk' of github.com:wordpress/gutenberg into update/im…
alexstine eb9ba86
Merge branch 'trunk' into update/image-describedby
carolinan 0cb78a0
CS: fix spacing after merge conflict
carolinan 30526b0
Another attempt to pass the native tests.
carolinan 19d0a3b
Avoid using the media id to create the id for aria-describedby, updat…
carolinan 2f4631d
rename deprecated fixture for consistency
carolinan d47f0c4
Update edit.native.js
carolinan bbac91d
Update index.native.js.snap
carolinan fcb3b30
Update initial-html.js
carolinan ace087e
Update edit.native.js
carolinan 8147467
Update initial-html.js
carolinan d487539
fix another copy paste error...
carolinan 15a875e
Update blocks-raw-handling.test.js.snap
carolinan 480dae1
Update blocks-raw-handling.test.js.snap
carolinan 3686d1d
Update edit.native.js
carolinan 9a4c728
Update edit.native.js
carolinan baa6d4d
Try to catch the exception when there is no URL
carolinan 2fcbeee
Update image.js
carolinan 8f6b4aa
Remove duplicate aria-describedby when the image is linked
carolinan a954da3
Update edit.native.js
carolinan 211a708
Merge branch 'trunk' into update/image-describedby
aristath 1e4e0ca
Merge branch 'trunk' into update/image-describedby
carolinan 8560685
Merge branch 'trunk' into update/image-describedby
aristath 531ffc6
Merge branch 'trunk' into update/image-describedby
carolinan bb597d2
try adding new fixture for image with alt text
carolinan f6aab6d
Update index.php
carolinan 418f446
Merge branch 'trunk' into update/image-describedby
carolinan 1c82f49
Update deprecated.js
carolinan 67ec32a
Update index.php
carolinan 21c462b
Use the WP_HTML_Tag_Processor
carolinan a2e3e21
Update index.php
carolinan 50c8ec8
Merge branch 'trunk' into update/image-describedby
carolinan 0e4d5eb
Merge branch 'trunk' into update/image-describedby
carolinan 16b7c1b
Merge branch 'trunk' into update/image-describedby
carolinan 7488a34
Merge branch 'trunk' into update/image-describedby
carolinan eb0b58c
Merge branch 'trunk' into update/image-describedby
carolinan 4d7fa46
Merge branch 'trunk' into update/image-describedby
carolinan 229c4c2
Merge branch 'trunk' into update/image-describedby
aristath 0085840
Merge branch 'trunk' into update/image-describedby
carolinan 09165b4
Merge branch 'trunk' into update/image-describedby
carolinan 5b18a64
Update gallery block native test snapshots to include aria-describedby
carolinan 0d6f9b8
Update gallery + image block native test snapshots to use aria-descri…
carolinan 6722153
Merge branch 'trunk' into update/image-describedby
carolinan 9101972
Merge branch 'trunk' into update/image-describedby
carolinan d2576f9
Merge branch 'trunk' into update/image-describedby
carolinan 6c52ae7
Update deprecated.js
carolinan 5cedc9c
Merge branch 'trunk' into update/image-describedby
carolinan 4368676
Try to solve merge clonflict
carolinan a15f7ce
Update index.php
carolinan 360fca6
Update the duotone supports in the deprecation
carolinan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These supports don't match those currently in the Image block.json on trunk.
The
__experimentalSelector
I don't think is critical but the duotone support was stabilized underfilter.duotone
. This should probably match that.