From #3111409: Add new Olivero frontend theme to Drupal 9.1 core as beta

 +++ b/core/themes/olivero/olivero.theme
@@ -0,0 +1,576 @@
+    $variables['table']['#header'][0]['data'] = [
+      '#type' => 'html_tag',
+      '#tag' => 'h4',
+      '#value' => $variables['element']['#title'],
+      '#attributes' => $header_attributes,
+    ];

Let's add a todo to change this after #3099026: Claro's preprocessing of field multiple value form's table header cell removes potential changes by others has landed

CommentFileSizeAuthor
#5 3165972-5.patch551 bytessd9121
#2 3165972-2.patch568 byteskostyashupenko

Comments

mherchel created an issue. See original summary.

kostyashupenko’s picture

Status: Active » Needs review
StatusFileSize
new568 bytes
mherchel’s picture

Status: Needs review » Needs work

This should be in the @todo format, and also wrap at 80 characters. See https://git.drupalcode.org/project/drupal/-/blob/9.0.x/core/themes/claro... as an example.

sd9121’s picture

Assigned: Unassigned » sd9121
sd9121’s picture

Assigned: sd9121 » Unassigned
Status: Needs work » Needs review
StatusFileSize
new551 bytes

Please review.

Thanks!

  • mherchel committed 31ec633 on 8.x-1.x authored by sd9121
    Issue #3165972 by sd9121, kostyashupenko: Add todo on form table header...
mherchel’s picture

Status: Needs review » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.