Skip to content

csedweek.org: fix styling by including its CSS again - #16148

Merged
breville merged 2 commits into
stagingfrom
fix-csedweek-styling
Jun 29, 2017
Merged

csedweek.org: fix styling by including its CSS again#16148
breville merged 2 commits into
stagingfrom
fix-csedweek-styling

Conversation

@breville

Copy link
Copy Markdown
Member

Addresses an issue introduced in #16039. It's easy to overlook that csedweek.org relies on code.org and then builds on top of it in a variety of ways. I'll add some eyes tests to cover csedweek.org shortly.

@wjordan

wjordan commented Jun 29, 2017

Copy link
Copy Markdown
Contributor

@breville I worked on #16153 yesterday which should address this issue while cleaning up the code a bit as well. Let me know what you think.

@tanyaparker tanyaparker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's merge this in to fix the site asap with today's DTP and then continue fixing with Will's refactoring.

@breville
breville merged commit b0a9886 into staging Jun 29, 2017
@breville
breville deleted the fix-csedweek-styling branch June 29, 2017 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants