Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.The header for entity update lists seems too large.
There is a CSS rule that supposed to handle this but it is not applied because the system-status-report__entry class is no longer used in the corresponding template.
| Comment | File | Size | Author |
|---|---|---|---|
| #17 | Screenshot from 2023-04-03 15-22-20.png | 31.75 KB | larowlan |
| #17 | Screenshot from 2023-04-03 15-20-47.png | 33.7 KB | larowlan |
| #7 | after-patch-applying.png | 47.56 KB | akshay kashyap |
| #3 | 3350387-3.patch | 1.04 KB | gauravvvv |
| status-report.png | 73.82 KB | chi |
Issue fork drupal-3350387
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
chi commentedComment #3
gauravvvv commentedI have updated the selector in the patch. please review
Comment #4
rinku jacob 13 commentedComment #5
rinku jacob 13 commentedComment #6
akshay kashyap commented@Gauravvvv Thanks for the work. I have applied your patch it's working fine
Comment #7
akshay kashyap commentedComment #8
akshay kashyap commentedComment #9
chi commented"system-status-report__entry__value" class looks weird. It doesn't follow BEM naming convention.
Comment #12
rohan-sinha commentedAssuming that the "system-status-report__entry__value" is a component or element of a larger system status report UI, I would name it using the BEM methodology as follows:
Block: system-status-report
Element: entry
Modifier: value
Thus, the complete BEM name for this element would be "system-status-report__entry--value".
Comment #13
chi commentedI think "value" is not a modifier here but element. That template initially had weird class names. Fixing them is out of scope of this issue.
Patch #3 looks good. The only concern here is that element "system-status-report__entry__value" has no parent component.
Comment #14
smustgrave commentedWould agree with #13 but since it's currently not using a parent think it should be safe.
#3 fixed the issue.
Comment #15
larowlanRemoving credit for the MR, as there was already an RTBC patch beforehand
Comment #16
larowlanConfirmed the existing template uses
system-status-report__entry__valueComment #17
larowlanNot granting credit for the screenshots, which would have been useful, but unfortunately don't show a h3 element
here's some before/after shots with a hacked in h3 that was in caps in the source (ie the capitalization isn't from CSS).
This seems like a simple misnamed selector in the file, so going to commit.
Comment #18
larowlanFixing typo
Comment #21
larowlanCommitted to 10.1.x and backported to 10.0.x and 9.5.x
Comment #23
drumm(Updating to resolve Drupal.org issue indexing issue, please disregard)