Bug 32155: Fix alignment of course details page

This patch eliminates some unnessesary CSS and converts the course details div into a fieldset.

To test:
1. Create or edit an existing course.
2. Search and add an instructor.
3. Note that the columns are misaligned: the name of the course instructor is shown first, then the label "Instructors" is shown next.
4. Add another three instructors.
5. Note that this pushes the labels for the next few fields to the right.
6. Apply patch
7. Try steps 1-4 again and confirm everthing looks and works correctly.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Lucas Gass 2022-11-14 21:07:21 +00:00 committed by Tomas Cohen Arazi
parent 737b4ec006
commit acfa184f78
Signed by: tomascohen
GPG key ID: 0A272EA1B2F3C15F

View file

@ -11,12 +11,6 @@
[% INCLUDE 'doc-head-close.inc' %]
[% FILTER collapse %]
<style>
#instructors {
float: left;
}
.instructor_line {
padding: 0 .1em;
}
.biblio-level-info {
background-color: #f3f3f3;
border-radius:5px;
@ -81,7 +75,7 @@
<div class="page-section">
<div class="rows">
<fieldset class="rows">
<ol>
<li><span class="label">Course name</span> [% course.course_name | html %]</li>
<li><span class="label">[% tp('Semester', 'Term') | html %]</span> [% AuthorisedValues.GetByCode( 'TERM', course.term ) | html %]</li>
@ -103,7 +97,7 @@
<li><span class="label">Student count</span> [% course.students_count | html %]</li>
<li><span class="label">Status</span> [% IF course.enabled == 'yes' %]Active[% ELSE %]Inactive[% END %]</li>
</ol>
</div>
</fieldset>
</div> <!-- /.page-section -->
[% IF course_reserves %]