Merge pull request #2656 from drgrice1/jitar-disabled-problem-no-jquery

Make JITAR disabled problems disabled without javascript.
openwebwork · Jan 28, 2025 · 3a18a63 · 3a18a63
2 parents 6fbdae6 + 643debd
commit 3a18a63
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 9 deletions.
diff --git a/htdocs/js/Problem/problem.js b/htdocs/js/Problem/problem.js
@@ -4,9 +4,4 @@
 		const bsToast = new bootstrap.Toast(toast, { delay: 5000 });
 		bsToast.show();
 	});
-
-	// Prevent problems which are disabled from acting as links
-	$('.problem-list .disabled-problem')
-		.addClass('disabled')
-		.on('click', (e) => e.preventDefault());
 })();
diff --git a/lib/WeBWorK/ContentGenerator/Problem.pm b/lib/WeBWorK/ContentGenerator/Problem.pm
@@ -797,9 +797,10 @@ sub siblings ($c) {
 			{
 				push(
 					@items,
-					$c->link_to(
-						$c->maketext('Problem [_1]', join('.', @seq)) => '#',
-						class                                         => $class . ' disabled-problem',
+					$c->tag(
+						'a',
+						class => $class . ' disabled',
+						$c->maketext('Problem [_1]', join('.', @seq))
 					)
 				);
 			} else {

diff --git a/templates/ContentGenerator/ProblemSet/problem_list_row.html.ep b/templates/ContentGenerator/ProblemSet/problem_list_row.html.ep
@@ -37,7 +37,7 @@
 			% )
 			% {
 				% # If the problem is jitar restricted, then show it greyed out.
-				<span class="<%= $linkClasses %> disabled-problem text-nowrap">
+				<span class="<%= $linkClasses %> text-nowrap">
 					<%= maketext('Problem [_1]', $problemNumber) %>
 				</span>
 			% } else {