From 161b230b6fc2647b1d1933047a2954e1748a8728 Mon Sep 17 00:00:00 2001 From: Glenn Rice Date: Tue, 8 Sep 2026 06:41:48 -0500 Subject: [PATCH 1/2] Rework the test problem header. Currently the message that reports if answers are recorded or not is the first thing in a problem in a test, and that also has the `id` of the problem. That `id` element is what is used as the anchor when a problem jump link is used or if entering the test directly to a problem from a link else where. Furthermore, in most of those cases when those fragment URLs are used, that recorded answer div is empty. This results in odd behavior such as that empty div being outlined. So move that div down after the actual problem header and wrap the problem header (and sub header) in a div that now contains the `id` of the problem. This is much more intuitive when one of these fragment URLs is used. Note that this needs #3203 to work better, and will conflict with #3202 (but the changes in #3202 that this conflicts with should be removed with this change). --- templates/ContentGenerator/GatewayQuiz.html.ep | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/templates/ContentGenerator/GatewayQuiz.html.ep b/templates/ContentGenerator/GatewayQuiz.html.ep index c54b90b414..78e3f8090d 100644 --- a/templates/ContentGenerator/GatewayQuiz.html.ep +++ b/templates/ContentGenerator/GatewayQuiz.html.ep @@ -617,10 +617,9 @@ % } %
- % # Show the jump to anchor. -
" tabindex="-1"><%= $recordMessage %>
- % # Output the problem header. -

<%= maketext('Problem [_1].', $i + 1) %>

+

" class="gw-problem-number" tabindex="-1"> + <%= maketext('Problem [_1].', $i + 1) %> +

% if ($c->{can}{showTemplateIds}) { <%= @@ -639,6 +638,9 @@ <%= $problems->[ $probOrder->[$i] ]->source_file %> % } + % if ($recordMessage) { +
<%= $recordMessage %>
+ % } % % my $instructor_comment = $c->get_instructor_comment($problems->[ $probOrder->[$i] ]); % if ($instructor_comment) { From 6bf702d26b11cfa2183867bdc93c5998fd95bd21 Mon Sep 17 00:00:00 2001 From: Glenn Rice Date: Fri, 11 Sep 2026 10:02:13 -0500 Subject: [PATCH 2/2] Add `outline: none` to the problem header. --- htdocs/js/GatewayQuiz/gateway.scss | 1 + 1 file changed, 1 insertion(+) diff --git a/htdocs/js/GatewayQuiz/gateway.scss b/htdocs/js/GatewayQuiz/gateway.scss index 06690c9c96..0b6dea7120 100644 --- a/htdocs/js/GatewayQuiz/gateway.scss +++ b/htdocs/js/GatewayQuiz/gateway.scss @@ -67,6 +67,7 @@ table.attemptResults { h2.gw-problem-number { display: inline-block; + outline: none; font-size: 16px; margin-right: 5px; line-height: 1.8;