18F / cg-deck

71%

master: 71%

LAST BUILD ON BRANCH jk-css-modules

branch: jk-css-modules

CHANGE BRANCH

x

Reset

• jk-css-modules
• 1.0.0
• 1.1.0
• 1.2.0
• 1.2.1
• concourse
• deploy
• deprecated
• govcloud-deploy
• govcloud-fix
• home-update
• jk-add-memory-units
• jk-css-cleanup
• jk-loading-bugs
• jk-marketplace-service-desc
• jk-no-apps
• jk-no-dev-ga
• jk-route-domain
• js-app-name-deploy
• js-fix-client-setup
• master
• ms-accessibility_table_scope
• ms-add_uaa_dashboard_icon
• ms-additional_space_initial_load_fixes
• ms-alphabetize_sidenav
• ms-cache_phantomjs2
• ms-change_deployment_names
• ms-change_go_names
• ms-content_on_pages
• ms-content_updates
• ms-continue_fixtures
• ms-fix_checkboxes
• ms-fix_space_page_loading
• ms-limit_current_user_actions
• ms-modify_nr_config
• ms-release-1.1.0
• ms-sidebar_context
• ms-update_cg_style-093
• ms-upgrade_npm
• ms-zero_mb_formatting
• notice
• notice-update
• release-1.2.0
• release-1.2.1
• staging
• staging-alpha
• travis
• travis-deploy

Build # 931

Build Type

push

travis-ci

Committed by jeremiak

Commit Message

Use single css file for all components

CSS modules is a cool idea, but our implementation got a bit
complex. We were importing the individual component files
from the style library, meaning that anybody working on a
component had to know not only that a class was in cloudgov-
style but also which file it was in. This was a lot of work
for something as simple as adding a class.

Now, all components just use the final, generated output css
from cloudgov-style so that any class in the library should
work pretty well in any component. I kept the .styler()
convention mostly in place to minimize changes.

The best illustration of how this changes improves code
legibility is in navbar.jsx right after render() begins.
Where we used to have to know which component file a class
was in, now we can simply use any class from the style
library.

Refs #350

Run Details

237 of 334 relevant lines covered (70.96%)

3.17 hits per line