ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

hide sequential order checkbox when first edited 

test plan:
* edit a context module with no requirements
* the checkbox for sequential order should not be shown
 until there are requirements

closes #CNVS-24094

Change-Id: Iae993d2f9f23713a33886872bd46ab29b3d0ee61
Reviewed-on: https://gerrit.instructure.com/65293
Tested-by: Jenkins
Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
QA-Review: Jahnavi Yetukuri <jyetukuri@instructure.com>
Product-Review: James Williams  <jamesw@instructure.com>
This commit is contained in:
James Williams 2015-10-21 13:40:33 -06:00
parent 1d7f9bc467
commit 3ebacdfac7
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
public/javascripts/context_modules.js
View File

@ -381,9 +381,9 @@ define([
$form.find(".completion_entry .no_items_message").show();
}
if ($module.find(".content .context_module_item .criterion.defined").length !== 0) {
$(".requirement-count-radio").children().show();
$(".requirement-count-radio").show();
} else {
$(".requirement-count-radio").children().hide();
$(".requirement-count-radio").hide();
}
var $requirementCount = $module.find('.pill li').data("requirement-count");
@ -926,7 +926,7 @@ define([
$option.find(".id").change();
$form.find(".completion_entry .criteria_list").append($pre).show();
$pre.slideDown();
$(".requirement-count-radio").children().show();
$(".requirement-count-radio").show();
$('#context_module_requirement_count_').change().focus();
});
$("#completion_criterion_option .id").change(function() {
@ -974,7 +974,7 @@ define([
$(this).remove();
// Hides radio button and checkbox if there are no requirements
if ($elem.html().length === 0 && $requirement.length !== 0) {
$(".requirement-count-radio").children().fadeOut("fast");
$(".requirement-count-radio").fadeOut("fast");
}
})
});