Page MenuHomePhabricator

Clean up _pages.scss
Closed, ResolvedPublic3 Estimated Story Points

Description

As a developer,
I want to be sure that the provided CSS is meaningful, applicable, and well-structured,
so that I can provide the least amount of styles to the user (for performance), can debug visual inconsistencies quickly and know where to find which styles

Acceptance Criteria:

  • All styles still in skins/cat17/src/sass/layout/_pages.scss are moved either to standalone subpages in skins/cat17/src/sass/layout/pages/ and included in _pages.scss or deleted.
  • Site looks like before (or "better").

Hints:

  • Compare with the HTMl from the 3rd party to see where the classes are/were used
  • You might need to add a page- prefix before some classes because we introduced the Twig page_identifier that automatically gets the prefix.
  • You might revert https://github.com/wmde/FundraisingFrontend/pull/1124 to see pages where the page_identifier is missing.

Related Objects

StatusSubtypeAssignedTask
ResolvedNone
ResolvedNone

Event Timeline

kai.nissen set the point value for this task to 3.Apr 10 2018, 1:51 PM

Seems like this was done. The file looks as described in the AC.