Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rewrite css in bem fashion #25

merged 14 commits into from Mar 7, 2016

Rewrite css in bem fashion #25

merged 14 commits into from Mar 7, 2016


Copy link

studioromeo commented Feb 14, 2016

Finally getting around to using BEM and SASS in the project. I'm using BEMIT here, because it looked quite nice and actually was quite helpful!

Also fixed a notable bug with the iframe which had been bugging me for a while. Before this PR the link would be fluid however once clicked a size would be set against the dynamically loaded iFrame meaning if the container changed in width the iFrame wouldn't follow it. Now the iFrame uses the same class as the link so it scales with its container.

Also as part of this PR the htmlproofer config has been adjusted to only report on 4xx errors instead so builds don't fail unnecessarily (eg because a server is currently experiencing 5xx errors). I did try parallelising the build however I didn't notice any performance benefits at this stage.

@studioromeo studioromeo force-pushed the bem branch 2 times, most recently from 758ffd8 to 262effb Feb 23, 2016
@studioromeo studioromeo closed this Mar 2, 2016
@studioromeo studioromeo reopened this Mar 2, 2016
@studioromeo studioromeo force-pushed the bem branch 3 times, most recently from 6cf0e2d to 240cabe Mar 7, 2016
Other errors we are not concerned about
studioromeo added a commit that referenced this pull request Mar 7, 2016
Rewrite css in bem fashion
@studioromeo studioromeo merged commit 7821970 into master Mar 7, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
@studioromeo studioromeo deleted the bem branch Mar 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.