Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.I just wrote a preprecessor to transform the ids on blocks to classes:
<?php
/*
* Implements hook_preprocess_block().
*/
function template_preprocess_block(&$variables) {
if (isset($variables['attributes']['id'])) {
$variables['attributes']['class'] = $variables['attributes']['id'];
unset($variables['attributes']['id']);
}
}
?>Having ids in the markup is against D8 standards, right?
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 2508357-6.patch | 649 bytes | joelpittet |
Comments
Comment #1
star-szrAdding a related issue, I can't recall if removing IDs was discussed on this one. I'm not saying this is a duplicate, small pieces are good (the issue I'm adding is over 2 years old and still open).
My suggestion for now would be to add a patch that removes the ID at the source and see what tests fail :)
Comment #2
star-szrAnd another one that might be worth a look that is talking about updating the IDs.
Comment #3
joelpittetThere is a requirement for a couple things I believe for ID's on block but no reason why we couldn't use a custom data attribute that gets added.
Eg. contextual links can inject a
data-contextual-id="search-block"on when it has permissions and is enabled.Comment #6
joelpittetCurious how testbot reacts to this. Maybe we could do this in a BC way with config and an update hook?
Comment #21
Liam MorlandYou can probably do this in
hook_preprocess_HOOK()forblock.