code-corps / code-corps-ember

Ember web application for Code Corps.
https://www.codecorps.org
MIT License
123 stars 78 forks source link

Use BEM for Styles #672

Open rileytaylor opened 8 years ago

rileytaylor commented 8 years ago

Problem

We want to standardize our stylesheets using BEM. This is a master issue to track all needed refactoring.

The lists below correspond to stylesheets. Each stylesheet (except those with _ in front of the name) correspond to a .hbs template file. General styles will need to be renamed to fit the naming conventions, which then need to be changed everywhere they exist in templates. Take one _name.scss file at a time. Create a separate issue and reference this master issue. Take advantage of git grep to find all references to that style. If you are making changes to styles within components or templates, feel free to create a grouped issue and pull request (as in #673) Once each issue and it's related PR's are complete, check off the files refactored on this list. Make sure you make changes in both the .scss and .hbs!

Files to Refactor

General

Templates

Components

rlueder commented 7 years ago

I'd like to help with the CSS refactoring efforts, I'm thinking _buttons.scss and any of the donation components still left (mainly because I spent time on them last night).

rlueder commented 7 years ago

I started looking into refactoring _colors.scss I'll have a PR later tonight.

marineb commented 7 years ago

Taking on _alerts.scss at #826

marineb commented 7 years ago

Taking on _code.scss with issue #829.

rlueder commented 7 years ago

I checked quite a few boxes on the list. I have four small integrations tests failing mainly due to renamed css classes, I'll fix those tomorrow at some point before moving on to projects and tasks.

samserif commented 7 years ago

Working on #673