Experimental result popover.

drgrice1 · Oct 18, 2023 · 655b0fa · 655b0fa
1 parent c42143f
commit 655b0fa
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 27 deletions.
diff --git a/lib/WeBWorK/ContentGenerator/Instructor/JobManager.pm b/lib/WeBWorK/ContentGenerator/Instructor/JobManager.pm
@@ -33,7 +33,6 @@ use constant FIELDS => [
 	[ started  => x('Started') ],
 	[ finished => x('Finished') ],
 	[ state    => x('State') ],
-	[ result   => x('Result') ]
 ];
 
 use constant SORT_SUBS => {
@@ -42,8 +41,7 @@ use constant SORT_SUBS => {
 	created  => \&byCreatedTime,
 	started  => \&byStartedTime,
 	finished => \&byFinishedTime,
-	state    => \&byState,
-	result   => \&byResult
+	state    => \&byState
 };
 
 sub initialize ($c) {
@@ -178,6 +176,5 @@ sub byCreatedTime  { return $a->{created}  <=> $b->{created} }
 sub byStartedTime  { return $a->{started}  <=> $b->{started} }
 sub byFinishedTime { return $a->{finished} <=> $b->{finished} }
 sub byState        { return $a->{state} cmp $b->{state} }
-sub byResult       { return lc $a->{result} cmp lc $b->{result} }
 
 1;
diff --git a/templates/ContentGenerator/Instructor/JobManager.html.ep b/templates/ContentGenerator/Instructor/JobManager.html.ep
@@ -100,10 +100,6 @@
 						<%= link_to maketext('State') => '#', class => 'sort-header',
 							data => { sort_field => 'state' } =%>
 					</th>
-					<th>
-						<%= link_to maketext('Result') => '#', class => 'sort-header',
-							data => { sort_field => 'result' } =%>
-					</th>
 				</tr>
 			</thead>
 			<tbody class="table-group-divider">
@@ -137,14 +133,28 @@
 									$jobs->{$jobID}{finished}, '', 'datetime_format_medium', $ce->{language}) =%>
 							% }
 						</td>
-						<td><%= $jobs->{$jobID}{state} %></td>
-						<td style="min-width:20em">
-							% if (ref($jobs->{$jobID}{result}) eq 'ARRAY') {
-								% for (@{ $jobs->{$jobID}{result} } ) {
-									<p class="m-0"><%= $_ %></p>
-								% }
-							% } else {
-								<%= $jobs->{$jobID}{result} // '' %>
+						<td>
+							<%= $jobs->{$jobID}{state} =%>
+							% if (defined $jobs->{$jobID}{result}) {
+								% content_for "result_$jobID", begin
+									% if (ref($jobs->{$jobID}{result}) eq 'ARRAY') {
+										% for (@{ $jobs->{$jobID}{result} } ) {
+											<p class="m-0"><%= $_ %></p>
+										% }
+									% } else {
+										<%= $jobs->{$jobID}{result} %>
+									% }
+								% end
+								<%= link_to '#', class => 'help-popup',
+									data => {
+										bs_toggle => 'popover',
+										bs_title => maketext('Result'),
+										bs_html => 'true',
+										bs_content => content("result_$jobID")->xml_escape
+									}, begin =%>
+									<i class="icon fa-solid fa-circle-question" data-alt="Result" aria-hidden="true">
+									</i>
+								<% end =%>
 							% }
 						</td>
 					</tr>

diff --git a/templates/HelpFiles/InstructorJobManager.html.ep b/templates/HelpFiles/InstructorJobManager.html.ep
@@ -47,17 +47,13 @@
 	<dt><%= maketext('State') %></dt>
 	<dd>
 		<%= maketext('The current state of the job.  This will be one of "inactive", "active", "finished", or '
-			. '"failed". If a job is "inactive" it means that the job has been added to the queue, but execution of '
-			. 'the job has not yet started. If a job is "active" it means that the job is currently being executed. '
-			. 'If a job is "finished" it means that the execution of the job has successfully completed. If a job is '
-			. '"failed" it means that the execution of job has completed, but there were errors in the execution of '
-			. 'the job.') =%>
-	</dd>
-	<dt><%= maketext('Result') %></dt>
-	<dd>
-		<%= maketext('The result set by the job when execution is completed.  This will be shown if the job is in the '
-			. '"finished" or "failed" states, and will contain information about what happened when the job was '
-			. 'executed.') =%>
+			. '"failed". If a job is "inactive" it means that the job has been added to the queue, but execution '
+			. 'of the job has not yet started. If a job is "active" it means that the job is currently being '
+			. 'executed. If a job is "finished" it means that the execution of the job has successfully '
+			. 'completed. If a job is "failed" it means that the execution of job has completed, but there were '
+			. 'errors in the execution of the job. If the job is in the "finished" or "failed" state, then there '
+			. 'will also be a popover containing information about what happened when the job was executed.') =%>
+		</p>
 	</dd>
 </dl>
 %