In #2193693: Improve theme description and screenshot alt-text for usability: we discussed the tags for drupal themes, and we couldn't find how they are used by Drupal.org.
As core themes set examples and expectations for other themes, including unused tags set an expectation for others to use tags, even if they have no use.
Remove tags from all core themes
| Issue category | Task, minor cleanup. |
|---|---|
| Issue priority | Normal because it's dead code. |
| Prioritized changes | Not a prioritized change. |
| Disruption | Not disruptive, this is just removing dead code. |
None
None
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | remove_tags_from_all-2471619-3.patch | 607 bytes | jeanfei |
Comments
Comment #1
lewisnymanComment #2
lewisnymanComment #3
jeanfei commentedI've removed the "tags" line in seven theme which was the only one with tags information.
Comment #4
fabianx commentedI am okay with that change if we don't use the tags anywhere, assigning to Cotter for final sign-off.
Comment #5
star-szrHmm…
So I ran:
Which led to #2193691: New methods, standards for describing Seven themes admin page.
From the issue summary there:
I don't think even if that was implemented on d.o it should be source from the .info file. Themes on d.o by definition are nodes, and can have a term reference field.
So I'm in favour of removing this metadata since it is not being used and IMO should not be used.
Comment #6
star-szrAdding beta evaluation.
Comment #7
alexpottCommitted 3a85c89 and pushed to 8.0.x. Thanks!
Thanks for adding the beta evaluation to the issue summary.