no_striping option for table rows is not respected

Created on 13 November 2018, over 5 years ago
Updated 13 March 2023, over 1 year ago

Problem/Motivation

#2329767: Move table classes from preprocess to templates broke per-row stiping.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Needs work

Version

10.1

Component
Theme 

Last updated 1 minute ago

Created by

🇺🇸United States tim.plunkett Philadelphia

Live updates comments and jobs are added and updated live.
  • Needs tests

    The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

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 Nikhil_110

    Attached patch against drupal 10.1.x

  • Status changed to Needs review over 1 year ago
  • Status changed to Needs work over 1 year ago
  • 🇺🇸United States smustgrave

    Still needs tests so moving to NW for that.

    #13 was a valid reroll but just FYI

    Starting March 2023, simple rerolls, rebases, or merges will no longer receive issue credit. Only rerolls that address a merge conflict will be credited, and the merge conflict that was resolved must be documented in the text of an issue comment.

    So a comment should of been added like "Tried applying patch #2 got
    error: core/themes/classy/templates/dataset/table.html.twig: No such file or directory"

    To receive credit for contributing to this issue, assist with other outstanding tasks or unaddressed feedback.
    See the issue credit guidelines for more information.

Production build 0.69.0 2024