Contrib.social

Remove classes from system templates f*.html.twig

Open on Drupal.org →
Created on 15 January 2015, almost 10 years ago
Updated 2 May 2024, 8 months ago

Problem/Motivation

Remove non essential classes from core - the classes are in classy, cores templates should be as clean as possible.
simpletest should be done with classy theme, not stark

Proposed resolution

See parent issue(s) and related issue(s) for details,

Twig Templates to modify

  • core/modules/system/templates/feed-icon.html.twig
  • core/modules/system/templates/field-multiple-value-form.html.twig
  • core/modules/system/templates/field.html.twig
  • core/modules/system/templates/fieldset.html.twig
  • core/modules/system/templates/form-element-label.html.twig
  • core/modules/system/templates/form-element.html.twig
  • core/modules/system/templates/form.html.twig

Remaining tasks

User interface changes

API changes

📌 Task
Status

Closed: outdated

Version

11.0 🔥

Component
Theme  →

Last updated 4 days ago

Created by

🇮🇳India Sivaji_Ganesh_Jojodae Chennai

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

    The change/bugfix cannot be fully demonstrated by automated testing, and thus requires manual testing in a variety of environments.

  • Needs screenshots

    The change alters the user interface, so before and after screenshots should be added to document the UI change. Make sure to capture the relevant region only. Use a tool such as Aviary on Windows or Skitch on Mac OS X.

  • Needs change record

    A change record needs to be drafted before an issue is committed. Note: Change records used to be called change notifications.

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.

  • Comment 11 months ago →
    🇳🇿New Zealand quietone
  • Status changed to Closed: outdated 8 months ago
  • Comment 8 months ago →
    🇺🇸United States bnjmnm Ann Arbor, MI

    This was filed ~10 months before Drupal 8 was released, any time during those 10 months would have been a fine time to take care of this.
    Drupal 8+ has now been around for 8.5 years, and removing the classes now would be incredibly disruptive and cause many regressions. The intentions here were good, but any benefits they'd have yeilded are trival compared to the problems that would be introduced. Closing so nobody winds up devoting any more time to something that I (Frontend Framework Manager) would not approve.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024