Allow FreeResponse in LevelGroup levels#7854
Conversation
The level title is displayed on standalone level pages but not on LevelGroup pages.
| } | ||
|
|
||
| .level-group-content { | ||
| overflow: hidden; |
There was a problem hiding this comment.
You can comment out this line to remove indentation and it should match the existing layout.
There was a problem hiding this comment.
I checked out the indentation and it works. Thanks for doing that. I might add additional indentation to get the textarea to line up with the multiple-choice answer indent, but can do that later.
There was a problem hiding this comment.
Might make sense to have .level-group { .number {} .content {} } except I worry about naming collisions, so don't worry about that for now :)
|
Is there a PR that includes a sample level and a sample inclusion in a .level_group? Would like to try this out. |
|
Pushed the migrated levels to a branch. You should be able to |
| - last_attempt ||= @last_attempt | ||
| - index ||= nil | ||
|
|
||
| - if index |
There was a problem hiding this comment.
Maybe a comment to explain to a newcomer that index indicates it's part of a LevelGroup?
|
lgtm |
In order to migrate open-response
TextMatchlevels to use theFreeResponselevel type,FreeResponselevels must work insideLevelGroupassessments.This change updates the
_free_response.html.hamlpartial so it can be hosted in the_level_group.html.hamlpartial.