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

Protective Behaviors quiz style fixes - plain quiz conflicts #74

Merged
merged 2 commits into from
Feb 24, 2015

Conversation

nikolas
Copy link
Member

@nikolas nikolas commented Feb 24, 2015

We need to address this issue:
ccnmtl/django-quizblock#35
so we don't have to deal with these problems!

@@ -3,6 +3,56 @@

{% getquestionresponse question as response %}

{% if 'protective-behaviors' in question.quiz.pageblock.css_extra %}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure you need all this...what issues are you seeing?

Copy link
Member Author

Choose a reason for hiding this comment

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

You're right - this part was unnecessary. I removed it.

@sdreher sdreher self-assigned this Feb 24, 2015
We need to address this issue:
    ccnmtl/django-quizblock#35
so we don't have to deal with these problems!
@sdreher
Copy link
Collaborator

sdreher commented Feb 24, 2015

I think protective behaviors can be dealt with via .css. From what I've seen so far, everything but likert can share the same single_choice template without much problem.

@nikolas
Copy link
Member Author

nikolas commented Feb 24, 2015

Alright - how should we resolve this then.. is it okay if I remove the class="caseanswercontainer" from the ol element? or should that be populated by question.quiz.pageblock.css_extra?

removing special casing
@sdreher sdreher removed their assignment Feb 24, 2015
nikolas added a commit that referenced this pull request Feb 24, 2015
Protective Behaviors quiz style fixes - plain quiz conflicts
@nikolas nikolas merged commit 2c0167b into master Feb 24, 2015
@nikolas nikolas deleted the pb-style-fixes branch February 24, 2015 21:04
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.

2 participants