Follow-up to #1342054: [META] Overhaul Bartik's code.
Problem/Motivation
- Bartik's template files need to be assessed and cleaned up of redundant markup, bad formatting and ID's.
- Bartik's CSS files need to follow Drupal's CSS Coding Standards.
Proposed resolution
For this issue we take "primary-menu.css" within Bartik in css/components/primary-menu.css plus any template file associated with the CSS and clean them up.
Remaining tasks
- Write a patch containing as much as the above as possible.
- Post a patch with screenshots.
- Visual review of a patch - check the secondary menu visually with and without patch applied. Take screenshots.
- Code review of a patch - check the code follows coding standards, suggest improvements if needed in a comment.
- Produce a new patch with improvements if needed.
User interface changes
None
API changes
None
Data model changes
None
Beta phase evaluation
| Issue category | Task because it is refactoring CSS and templates in Bartik |
|---|---|
| Issue priority | Not critical because Bartik functions fine we are just doing cleanup tasks |
| Unfrozen changes | Unfrozen because it only changes CSS and markup |
| Prioritized changes | The main goal of this issue is usability of the Bartik's codebase |
| Disruption | non-Disruptive as it is just changing markup and CSS |