Table row colors

Created on 19 October 2022, about 2 years ago
Updated 4 July 2023, over 1 year ago

Problem/Motivation

In 1.1.5 (Bootstrap 5.1) table row colors supported, for example, the class color-success; however in 2.0.2 (Bootstrap 5.2) it seems they must use table-success for the color to appear.

Steps to reproduce

View the missing table row colors at /admin/reports/updates

Proposed resolution

Either add way to change color-X to table-X for tables.
Or add color-X support back for tables.

Remaining tasks

Decide how to fix.

User interface changes

Table row colors are missing.

API changes

None.

Data model changes

None.

🐛 Bug report
Status

Needs review

Version

3.0

Component

Code

Created by

🇬🇧United Kingdom mustanggb Coventry, United Kingdom

Live updates comments and jobs are added and updated live.
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.

  • 🇮🇳India ravi kant Jaipur

    It working in latest version.
    It can confirm by creating a table or by installing Twbstools module. Install Twbstools module, enable and navigate '/styleguide'.

  • Assigned to Jay Jangid
  • Issue was unassigned.
  • 🇮🇳India Jay Jangid

    I have also tested it in my local system (drupal 9.5) as per @ravi kant.
    There are no issues with the table CSS.
    See the screenshot for reference.
    Thank you.

  • 🇬🇧United Kingdom mustanggb Coventry, United Kingdom

    Has anyone followed the reproduce steps yet?

    i.e. Does the table at /admin/reports/updates have the correct colors?

  • Assigned to abyss
  • Status changed to Active over 1 year ago
  • 🇺🇦Ukraine abyss

    This problem does exist if you install Bootstrap5 as an admin theme. So if no one minds, I will try to fix this problem.

  • @abyss opened merge request.
  • Issue was unassigned.
  • Status changed to Needs review over 1 year ago
  • 🇺🇦Ukraine abyss

    Updated the table row colors.

  • 🇬🇧United Kingdom mustanggb Coventry, United Kingdom

    It looks like you went with the second option: Or add color-X support back for tables.

    Can I ask what the problem with the first option was, because if there is a choice of approaches this seems to be preferable as it can just use the built-in Bootstrap CSS rather than requiring additional special case CSS: Either add way to change color-X to table-X for tables.

Production build 0.71.5 2024