Bug 38347: Fix style of sidebar form submit button on bookings to collect page

This patch fixes the markup of the sidebar form on the bookings to
collect page, and adds the correct "btn btn-primary" class to the submit
button.

NOTE: This patch contains indentation changes, so please ignore
whitespace when viewing the diff.

To test, apply the patch and go to Circulation -> Bookings to collect.

In the sidebar form, confirm that the submit button has the same yellow
"primary" style as other submit buttons, and that it fills the width of
the sidebar.

Sponsored-by: Athens County Public Libraries
Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Lisette Scheer <lisette@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
This commit is contained in:
Owen Leonard 2024-12-02 12:53:05 +00:00 committed by Katrin Fischer
parent 8557a5aef1
commit 6219271cd0
No known key found for this signature in database
GPG Key ID: 0EF6E2C03357A834
1 changed files with 48 additions and 50 deletions

View File

@ -52,59 +52,57 @@
<!-- Filters & Navigation -->
<div class="col-md-2 order-sm-2 order-md-1">
<aside>
<div id="filters">
<form id="bookingsf">
<fieldset class="brief">
<h4>Refine results</h4>
<ol>
<li>
<label for="holding_library">Holding library:</label>
<select name="holding_library" id="holding_library">
[% SET libraries = Branches.all( only_from_group => 1 ) %]
[% IF libraries.size != 1 %]
<option value="">Any</option>
<form id="bookingsf">
<fieldset class="brief">
<h4>Refine results</h4>
<ol>
<li>
<label for="holding_library">Holding library:</label>
<select name="holding_library" id="holding_library">
[% SET libraries = Branches.all( only_from_group => 1 ) %]
[% IF libraries.size != 1 %]
<option value="">Any</option>
[% END %]
[% FOREACH l IN libraries %]
[% IF (l.branchcode == branchcode) %]
<option value="[% l.branchcode | html %]" selected="selected">[% l.branchname | html %]</option>
[% ELSE %]
<option value="[% l.branchcode | html %]">[% l.branchname | html %]</option>
[% END %]
[% FOREACH l IN libraries %]
[% IF (l.branchcode == branchcode) %]
<option value="[% l.branchcode | html %]" selected="selected">[% l.branchname | html %]</option>
[% ELSE %]
<option value="[% l.branchcode | html %]">[% l.branchname | html %]</option>
[% END %]
[% END %]
</select>
</li>
<li>
<label for="pickup_library">Pickup library:</label>
<select name="pickup_library" id="pickup_library">
[% SET libraries = Branches.all( only_from_group => 1 ) %]
[% IF libraries.size != 1 %]
<option value="">Any</option>
[% END %]
[% FOREACH l IN libraries %]
[% IF (l.branchcode == branchcode) %]
<option value="[% l.branchcode | html %]" selected="selected">[% l.branchname | html %]</option>
[% ELSE %]
<option value="[% l.branchcode | html %]">[% l.branchname | html %]</option>
[% END %]
</select>
</li>
<li>
<label for="pickup_library">Pickup library:</label>
<select name="pickup_library" id="pickup_library">
[% SET libraries = Branches.all( only_from_group => 1 ) %]
[% IF libraries.size != 1 %]
<option value="">Any</option>
[% END %]
[% FOREACH l IN libraries %]
[% IF (l.branchcode == branchcode) %]
<option value="[% l.branchcode | html %]" selected="selected">[% l.branchname | html %]</option>
[% ELSE %]
<option value="[% l.branchcode | html %]">[% l.branchname | html %]</option>
[% END %]
[% END %]
</select>
</li>
<li>
<label for="from">Starts after: </label>
<input type="text" size="10" id="from" name="from" value="[% from | html %]" class="flatpickr" data-date_to="to"/>
</li>
<li>
<label for="to">Starts before: </label>
<input type="text" size="10" id="to" name="to" value="[% to | html %]" class="flatpickr"/>
</li>
</ol>
[% END %]
</select>
</li>
<li>
<label for="from">Starts after: </label>
<input type="text" size="10" id="from" name="from" value="[% from | html %]" class="flatpickr" data-date_to="to"/>
</li>
<li>
<label for="to">Starts before: </label>
<input type="text" size="10" id="to" name="to" value="[% to | html %]" class="flatpickr"/>
</li>
</ol>
</fieldset>
<fieldset class="action">
<input type="submit" name="run_report" value="Submit" class="btn btn-primary" />
</fieldset>
</form>
<fieldset class="action">
<input type="submit" name="run_report" value="Submit" class="submit"/>
</fieldset>
</fieldset>
</form>
</div>
[% INCLUDE 'circ-nav.inc' %]
</aside>
</div>