Contrib.social

Do not use "class" value on both "class" and "more_wrapper_class"

Open on Drupal.org →
Created on 28 January 2025, 2 days ago

Problem/Motivation Currently, smart trim uses the "class" setting value on both the "class" and "more_wrapper_class". Which leads to not optimal HTML-Output:

Instead, the wrap_class setting value should be used for the wrapper.

Steps to reproduce

Proposed resolution

Fix the problem.

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Active

Version

2.0

Component

Code

Created by

🇩🇪Germany Grevil

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @Grevil
  • Comment 2 days ago →
    nidhish
  • Comment 2 days ago →
    🇩🇪Germany Grevil
  • Comment 2 days ago →
    🇩🇪Germany Anybody Porta Westfalica

    @Maintainer: This is a potential breaking change and we're not sure if it's really a bug or intended behavior, but seems a bit strange that both elements have the same class. Still these are different HTML elements, so might be expected and by design.

    Feel free to close this "by design" if it's correct. We just saw that same class in code on both elements.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024