I am wondering why the heck my body fields get this class, after a grep i found it inside the base theme.

Why is this class added in the .theme file (ln. 141)?

It's a pain to remove it in the child .theme file (array_search ...).

If this class is still needed for some reason, shouldn't we add it inside a field--node--body.html.twig? Much easier to override and probably less performance impact?

Comments

thomas.frobieter created an issue. See original summary.

hongpong’s picture

Somewhat related thing i found today: #2941157: custom class does not appear on body field from field_formatter_class. i would concur with you it is better to add on the twig side if anywhere. If this term isn't a special zurb foundation thing i think it should be removed.

serg2’s picture

Going through the v6 foundation css I cannot find "text-secondary" so I think it maybe a left over from v5 or something, either way not required by ZF.

tanmaykadam’s picture

Status: Active » Needs review
StatusFileSize
new610 bytes

I have added patch to remove the class "text-secondary" from teasers.

neslee canil pinto’s picture

Removed class text-secondary from teaser mode

philosurfer’s picture

#4 and #5 are the same.

I can confirm #4 works.
No longer seeing class applied.

philosurfer’s picture

Status: Needs review » Reviewed & tested by the community

  • HongPong committed 430a745 on 8.x-6.x authored by tanmaykadam
    Issue #2940608 by Neslee Canil Pinto, tanmaykadam, philosurfer, thomas....
hongpong’s picture

Status: Reviewed & tested by the community » Fixed

Thank you everyone for your contributions here!

Status: Fixed » Closed (fixed)

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