Some discussion about -ms-high-contrast over here too:
https://www.drupal.org/project/drupal/issues/3327405
📌
Potentially remove -ms-high-contrast
Needs work
Came across this issue:
https://www.drupal.org/project/drupal/issues/3167032 →
, talking about -ms-high-contrast too.
Yeah, can confirm the CKEditor5 version of the function `process` does the trick. It's not only or --. Some other html might not be accepted as wel like <br> (without trailing slash).
Came across this issue and used the patch. Not because translation directory wasn't configured properly but just because it couldn't download with a drush locale:update:
ddev drush locale:update
> [error] Unable to download translation file https://ftp.drupal.org/files/translations/all/admin_toolbar/admin_toolbar-3.5.1.nl.po.
> [warning] filesize(): stat failed for //admin_toolbar-3.5.1.nl.po locale.bulk.inc:203
> [warning] fopen(translations://admin_toolbar-3.5.1.nl.po): Failed to open stream: "Drupal\locale\StreamWrapper\TranslationsStream::stream_open" call failed PoStreamReader.php:154
> [error] TypeError: fgets(): Argument #1 ($stream) must be of type resource, bool given in fgets() (line 248 of /var/www/html/web/core/lib/Drupal/Component/Gettext/PoStreamReader.php) #0 /var/www/html/web/core/lib/Drupal/Component/Gettext/PoStreamReader.php(248): fgets(false)...Patch saved my day. Rather have a broken translation than trying to figure out why it's unable to download translation. Agree with #34.
Looks like flash isn't used in the code anymore. Lets close this issue.
Agree with the suggention of Klonos. Remove the hidden span and use aria-selected. Following the MDN Web Docs https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/ta... they recommend to use that. Also best practice to use button but perhaps that is outside of this issue scope.
Sounds valid, can you confirm its working?
Nice, Thanks @jweowu! Think this issue can be closed now? Agree @tm01xx?
@narendra.rajwar27 Wouldn't it be better to just add your own patch to your specific project with this specific case instead of keeping a merge request open? Or am I getting your intention wrong.
More info on applying a local patch on a module:
Maybe consider not only look at paths but somewhere you can define roles in which case other rules are applied. Most easy fix would be to just add unsafe inline when a user role is in the defined roles.
@vensires Just for understanding, looking at the if (!str_starts_with(PHP_OS, 'WIN')) in core. It's always connected with a directory/file permission command. If you search for str_replace(DIRECTORY_SEPARATOR, '\\' they don't do the OS check. Is it necessary?
Housekeeping: Added 4 screenshots.
I've created the Merge Request. Can someone review it? https://git.drupalcode.org/project/drupal/-/merge_requests/9648
Going to work on this issue on DrupalCon Barcalona 2024. Chris Darke was my mentor. Discussed this issue with Ben. Going to start a new branch and start from there.
Just thinking.. In /web/core/misc/dialog/off-canvas/css/button.css the #drupal-off-canvas-wrapper .button is set to a width: 100%;.
If the dialog is on the right side of the (desktop) screen I guess it's designed to be as wide as the dialog. If the dialog is at the top or at mobile, it indeed looks a bit strange. I think it would look strange in all themes. Wouldn't it then be better to fix this in the dialog off-canvas button css file and maybe remove the width: 100% over there, or alternatively check if dialog is at left/right side and set the width to 100%?
Thanks for fast merging!
Groendijk → created an issue.
Wouldn't be use Drupal\search_api\LoggerTrait; use LoggerTrait; be more in line with the other code?