Page MenuHomePhabricator

Remove unused CSS classes
Closed, ResolvedPublic

Description

There are some parts that have been removed (related to old language search, old <button> etc), we need to remove the CSS too.

Note: we may also remove the language-search.handlebars template.

Event Timeline

Change 342289 had a related patch set uploaded (by JGirault):
[wikimedia/portals] Remove pieces of CSS not used on wikipedia.org

https://gerrit.wikimedia.org/r/342289

I think there's a lot more CSS that can be removed here.
There's a nifty chrome devtools feature that highlights what it thinks is unused CSS. These results need manual review because it picks up things like the 'no-js' and typeahead styles as unused, but I think it provides a good starting point.

Screen Shot 2017-03-11 at 12.28.10 PM.png (1×2 px, 444 KB)

@Jdrewniak it's true, I know there are more, this is a long and intense and cautious work in order to clean our CSS.

I'd rather do it iteratively. Keep cleaning stuff as we find it. Merge patches if they are good to be merged. And keep cleaning. There is some CSS classes with pure-buttons that I'd like to follow with a discussion with @Volker_E because some of it could be made available to the community, via the new style guide for instance.

debt triaged this task as Medium priority.Mar 15 2017, 12:30 AM

Change 342289 merged by jenkins-bot:
[wikimedia/portals@master] Remove pieces of CSS not used on wikipedia.org

https://gerrit.wikimedia.org/r/342289

debt removed a project: Patch-For-Review.
debt moved this task from Needs code review to Backlog on the Discovery-Portal-Sprint board.
debt subscribed.

Moving this to the backlog to work on when we begin automating the portal update code.

Change 383545 had a related patch set uploaded (by Jdrewniak; owner: Jdrewniak):
[wikimedia/portals@master] Removing unused CSS from www.wikipedia.org portal

https://gerrit.wikimedia.org/r/383545

Change 383545 merged by jenkins-bot:
[wikimedia/portals@master] Remove unused CSS & templates from www.wikipedia.org portal

https://gerrit.wikimedia.org/r/383545

This comment was removed by Jdrewniak.
debt claimed this task.