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

added problem type with 0/undefined slope; refactored out embedded JS #6124

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@stephjang
Contributor

stephjang commented Nov 11, 2011

Trello bug title

  • Add problem type to slope of a line exercise

Changes

  • Added problem type with 0/undefined slope
  • Refactored out embedded JS in
  • Added data-weight=2 to the existing exercises and data-weight=1 to the new one

Not changed

  • Kept the problem options un-randomized, due to referencing by array index within the Hints
@marcia

This comment has been minimized.

Show comment
Hide comment
@marcia

marcia Nov 11, 2011

Contributor

Great job refactoring my suboptimal code :)

I made a few whitespace changes here 35c0c45 and then I swapped out "infinity" for "undefined" here 7090569. Thanks so much, and I look forward to more things coming my way from you!

Contributor

marcia commented Nov 11, 2011

Great job refactoring my suboptimal code :)

I made a few whitespace changes here 35c0c45 and then I swapped out "infinity" for "undefined" here 7090569. Thanks so much, and I look forward to more things coming my way from you!

@marcia marcia closed this Nov 11, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment