This child issue will tackle most of the Views module templates. views-view.html.twig is handled in
After the removal of CSS classes from the core templates, many unnecessary HTML tags were left behind. They were not removed in the process of removing classes and other work, because removing them requires paying particular attention to any affect on the output.
Beta phase evaluation
|Issue priority||Normal because nothing is broken.|
|Unfrozen changes||Unfrozen because it only changes templates and possibly CSS which are both unfrozen.|
|Prioritized changes||The main goal of this issue is to improve themer experience.|
|Disruption||There should be minimal disruption.|
Go through each core template and remove any unnecessary markup. This will require looking at the output to see if things shift around in unwanted ways. Various CSS files may also need checking. For example, if there is CSS like
.some_class > div > .some_other_class the CSS should be adjusted, or deleted.
- divs and spans with no additional attributes should be removed mercilessly unless they justify their existence.
- Semantic tags (footer, article, etc.) should be retained even if they have no attributes.
User interface changes
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 97,304 pass(es). View
|#27||interdiff.txt||2.31 KB||Manuel Garcia|
|#27||remove_unnecessary-2497455-27.patch||6.35 KB||Manuel Garcia|
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 95,725 pass(es), 5 fail(s), and 0 exception(s). View
|#12||interdiff.txt||1.26 KB||Manuel Garcia|
|#12||remove_unnecessary-2497455-12.patch||4.04 KB||Manuel Garcia|
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 95,675 pass(es), 10 fail(s), and 1 exception(s). View