ram4nd → created an issue. See original summary → .
Doesn't seem like you missed anything.
Tested that the patch works and solves the error on "Status report" page.
It worked, I simplified the if condition. And I added position support as well.
Simply converted logout page to confirmation form. What else should be backported/implemented? And then it needs tests too.
ram4nd → created an issue. See original summary → .
If I remember correctly, I backported it from D11 not D10.
Why is this marked fixed?
Replaced by own code with back port from Drupal 11. https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Component%21Util...
I also reproduced this on Drupal 7 and opened a child issue. I used D7 Lando instance with PHP 8.3 and "Error messages to display" is set to "None".
Needs to be merged to D7 too
As per #3
Works, but produced a notice.
- The initial focus should be on the dialog container with role="dialog".
- When the dialog is visible and the focus has been moved on it, the keyboard focus must remain within modal dialogs when open.
Yes I am still interested.
- Add aria-labelledby to the dialog.
- Modal title should be heading not div.
- Add role close to close link(button).
- Add role dialog to container. Also add tabindex -1 so it can be focused by js and thus announced by screen reader.
- Fix keyboard detection, there is no "e.key" value Space.
Marking RTBC due to #16
Focus getting lost when modal dialog is closed by using Close button.
Add if condition to validate that "top" is set on sticking.
Go through parents and their siblings to get elements that have been made sticky. I think this is better approach than is used with .is-sticky where distance from top is calculated.
Needs a correct patch.