- ๐บ๐ธUnited States andyg5000 North Carolina, USA
Warning: The patch in #11 can cause issues with media_library widget. It causes the ajax/form-submit logic to break when clicking "apply filters" in the media library widget on a node form.
The error is:
Path: /admin/content/media-widget/image?name=sdf&sort_by=created. Drupal\Core\Http\Exception\CacheableAccessDeniedHttpException: The opener ID parameter is required and must be a string. in Drupal\Core\Routing\AccessAwareRouter->checkAccess() (line 115 of /var/www/html/docroot/core/lib/Drupal/Core/Routing/AccessAwareRouter.php). - First commit to issue fork.
- ๐ฉ๐ชGermany Anybody Porta Westfalica
Would this improve the situation maybe? โจ Support tippy.js by submodule Active
- ๐ฉ๐ชGermany Anybody Porta Westfalica
Thank you very much for the implementation, folks!
Now I'm wondering if we should really solve that on our own or better use a more accessibiliy friendly tooltip library? What do you think and is one out there?
- @anybody opened merge request.
- ๐ฆ๐บAustralia silverham
Added combined patch over at: https://www.drupal.org/project/entity_browser/issues/3036009#comment-157... โจ Configurable checkbox label RTBC
@see โจ Configurable checkbox label RTBC
- ๐ฆ๐บAustralia silverham
Attached is combined patch with ๐ Cannot add a label to the the checkbox column (Entity browser bulk select form) Needs review
- ๐ฆ๐บAustralia silverham
Closing as duplicate of โจ Configurable checkbox label RTBC
- ๐ฆ๐บAustralia silverham
Similar patch against latest 8.x-2.x version (note fields are different) but with
- Can use long text with HTML now.
- Has optional "CSS no-wrap" option. - ๐ฆ๐บAustralia silverham
Missed CSS file, causing crash + use HTML escape on label instead of just views built-in XSS admin function.
- Issue created by @silverham
The Needs Review Queue Bot โ tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide โ to find step-by-step guides for working with issues.
- @mariewakeman opened merge request.
Automatically closed - issue fixed for 2 weeks with no activity.
- ๐ช๐จEcuador jwilson3
okay, right. Color contrast does make sense to keep in scope here, Liam. Thanks.
- ๐ซ๐ฎFinland simohell
The current MR seems to add the checkbox twice on admin/content - once before and once after the toggle for in a narrow view toggling on and off the low priority column. The order of the two toggles is something to be decided. The toggle seems to be missing admin/content/blocks, admin/content/files or admin/content/media - the last of which also has an alternative grid-view that doesn't need one.
On permissions page the checkbox is also doubled, but there are no other controls in between.
I think we should have do both the toggles with similar implementation. Now one is a checkbox and the other is a button. Styling can then be decided per theme if one would like to use a pin-icon for sighted users etc.
- First commit to issue fork.
This change adds "layout--content-medium" to the element in the Olivero pager.html.twig file.
Not sure why it was added, but it messed up the layout of my site on a views page with pagination. I had to override the template and remove the class.https://git.drupalcode.org/project/drupal/-/commit/197d8bae7ddcab6093d30...
Leaving a comment here instead of opening a new ticket. Not sure if this is a bug, but some one else out there might be dealing with the same problem when updating to 10.3
- ๐บ๐ธUnited States katannshaw
Added "Related WCAG success criteria and techniques" to issue description to provide context and clarity.
- ๐บ๐ธUnited States smustgrave
Issue summary still appears to have the old screenshot. Donโt need to keep deleted text either
- ๐ฎ๐ณIndia Chandansha
I have tested MR 8816. Now unnecessary link tag removed.
I moved it to RTBC.
THANKS!! - ๐บ๐ธUnited States smustgrave
Summary appears to be outdated, example the "Apply to selected items" button no longer is there
Proposed solution appears to be missing.
I did test the "Filter" button and it passed contrast.
- ๐บ๐ธUnited States smustgrave
Re-ran failure for the editor module and appeared to be random.
Believe all feedback for this one has been addressed.
- ๐ซ๐ฎFinland simohell
Here is a new article about the topic:
Exploring the challenges in creating an accessible sortable list (drag-and-drop) by Kenfall Gassner from Girhub
https://github.blog/2024-07-09-exploring-the-challenges-in-creating-an-a...