Olivero: use multi-class array for adding multiple classes in form--search-block-form.html.twig template

In drupal/core/themes/olivero/templates/form--search-block-form.html.twig file, We are adding multiple classes by single array.
<form{{ attributes.addClass('search-form', 'search-block-form') }}>. We should add multiple classes by multi-class array.

CommentFileSizeAuthor
#5 image (5).png20.08 KBlibbna
#5 image (4).png30.68 KBlibbna
#2 3226016-2.patch601 bytesgauravvvv

Comments

Gauravmahlawat created an issue. See original summary.

gauravvvv’s picture

gauravvvv commented
Status: Active » Needs review
StatusFileSize
new 3226016-2.patch601 bytes

I have provided a patch which adding classes by multi class array.

Please review.

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.0-rc1 was released on November 26, 2021, which means new developments and disruptive changes should now be targeted for the 9.4.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

cilefen’s picture

cilefen commented
Category: Support request » Task

Maybe this is a task.

libbna’s picture

libbna commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new image (4).png30.68 KB
new image (5).png20.08 KB

reviewed patch #2 and classes has been added successfully by multi class array.

gauravvvv’s picture

gauravvvv commented

  • lauriii committed 17e8a50 on 10.0.x 
    Issue #3226016 by Gauravmahlawat, Libbna: Olivero: use multi-class array...

  • lauriii committed 79a302d on 9.5.x 
    Issue #3226016 by Gauravmahlawat, Libbna: Olivero: use multi-class array...

  • lauriii committed b06d950 on 9.4.x 
    Issue #3226016 by Gauravmahlawat, Libbna: Olivero: use multi-class array...
lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
commented
Status: Reviewed & tested by the community » Fixed

Committed 17e8a50 and pushed to 10.0.x. Cherry-picked to 9.5.x and 9.4.x. Thanks!

Status: Fixed » Closed (fixed)

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