Use Attribute::addClass() instead of CSS class concatenation in template_preprocess_views_view_table()

Created on 20 August 2015, almost 9 years ago
Updated 19 October 2023, 8 months ago

Follow-up to #2502089: Remove SafeMarkup::set() in template_preprocess_views_view_table()

Follow-up to #2280965: [meta] Remove every SafeMarkup::set() call

Problem/Motivation

@MauPalantir noticed there was a bunch of class concatenation done in template_preprocess_views_view_table()

Proposed resolution

Use Attribute() object to addClass() and/or move the class building to the template.

Remaining tasks

Manual testing steps

User interface changes

N/A

API changes

N/A

📌 Task
Status

Needs work

Version

11.0 🔥

Component
Views 

Last updated 1 minute ago

Created by

🇨🇦Canada joelpittet Vancouver

Live updates comments and jobs are added and updated live.
  • VDC

    Related to the Views in Drupal Core initiative.

  • Needs manual testing

    The change/bugfix cannot be fully demonstrated by automated testing, and thus requires manual testing in a variety of environments.

  • Novice

    It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.69.0 2024