[Koha-bugs] [Bug 32071] Consistent classes for primary buttons: Catalog

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32071

David Nind <david at davidnind.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |david at davidnind.com

--- Comment #2 from David Nind <david at davidnind.com> ---
Created attachment 143300
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143300&action=edit
Screenshots - where some changes are required

I think these two things need looking at:

1. catalogue/moredetail.tt (items section for a record): alignment issue with
'Update' button for public and non-public note fields. See document with before
and after screenshots attached.

2. catalogue/search-history.tt: Line 182 of catalogue/search-history.tt needs
updating as well for the previous sessions for authority records, similar to
previous sessions for catalog records. See document with screenshot attached.

Testing notes (using KTD).

1. These changed templates accessed from:
   Details view for a record
   Search results from a normal catalog search
   The cart
   User account menu (search history)

2. I tested by:
   - Reviewing the patch - <input type="submit"... becomes <input type="submit"
class="btn btn-primary"...
   - Listing the templates in koha-tmpl/intranet-tmpl/prog/en/modules/basket/
and koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/, and splitting into two
lists - those changed and those not changed
   - Working out how to access them
   - Creating screenshots and putting them in a document (I haven't attached
this, but am happy to)
   - Applying the patch (+ flush_memcached, restart_all, and cleared browser
cache)
   - Going through the list of files again: compared to original screenshot
taken, then noted whether changed or not
   - Noting any questions

3. Files changed:
   - basket/downloadcart.tt (No change. I couldn't find a link on the UI to get
to this form, was able to access directly at
http://127.0.0.1:8081/cgi-bin/koha/basket/downloadcart.pl, called from
basket/downloadcart.pl)
   - basket/sendbasketform.tt (No change. Add some items to the cart. View the
cart > Select all > Send.)
   - catalogue/advsearch.tt (Changed - search button now in orange. From the
menu bar: Search > Advanced search.)
   - catalogue/detail.tt (Not determined. I couldn't figure out how to trigger
or enable the "Download record" form/button.)
   - catalogue/itemsearch.tt (Changed - search button now in orange. Search
(from the navigation bar at top of screen) > Item search).)
   - catalogue/moredetail.tt (Changed - Set status and Update buttons are
larger. From the details page for a record select the 'Items' section.)
   - catalogue/results.tt (Minor change - search button is slightly bigger.
Search the catalog, under the search results is 'Search withing results'.)
   - catalogue/search-history.tt (Changed - 'Delete' is now an orange button,
dulled when no entries are selected, bright when entries are selected. Enable
EnableSearchHistory. Do some catalog and authority searches. Log out and then
log back in. Do some more catalog and authority searches. From the top
navigation bar go to your user account drop down list in the top right-hand
corner and select 'Search history'). 

4. Files not changed:
   - basket/basket.tt (Add some items to the cart. View the cart.)
   - basket/sendbasket.tt (This is the template for the body of the email
message sent when sending a cart.)
   - catalogue/imageviewer.tt (Enable LocalCoverImages, upload a cover image
for a record, triggered by clicking on the image displayed for the record and
under the images tab.)
   - catalogue/ISBDdetail.tt (View the ISBD section for a record.)
   - catalogue/issuehistory.tt (View the checkout history section for a
record.)
   - catalogue/itemsearch_csv.tt (Used to export details of items from Search >
Item search > [enter search term] > search results page.) 
   - catalogue/itemsearch_json.tt (Used to export details of items from Search
> Item search > [enter search term] > search results page.)
   - catalogue/labeledMARCdetail.tt (Enable viewLabeledMARC, view the labeled
MARC section for a record.)
   - catalogue/MARCdetail.tt (View the MARC section for a record.)
   - catalogue/showmarc.tt (Click on 'Show' for 'MARC preview' in the normal
details page for a record.)
   - catalogue/stockrotation.tt (Enable StockRotation and go to Cataloging >
Stock rotation. Add a new rota. Add a stage to the rota. Add an item to the
rota (Manage > Items > Add an item using barcode). View the rota section for a
record.)

5. File count check:
   - Number of files in folder - basket: 4
   - Number of files listed - basket: 4 (only 2 changed in this patch)
   - Number of files in folder -  catalogue: 15
   - Number of files listed - catalogue: 15 (only 6 changed in this patch)

Questions/notes: 

1. basket/downloadcart.tt: I couldn't find a link on the UI to get to this
form, was able to access directly at
http://127.0.0.1:8081/cgi-bin/koha/basket/downloadcart.pl, called from
basket/downloadcart.pl)

2. catalogue/detail.tt: I couldn't figure out how to trigger or enable the
"Download record" form/button - I see a form that may be it that flashes up
very briefly under the tabs section when viewing the details page for a record.

3. catalogue/moredetail.tt: alignment issue with 'Update' button for public and
non-public note fields. See document with before and after screenshots
attached.

4. catalogue/search-history.tt: Line 182 of catalogue/search-history.tt needs
updating as well for the previous sessions for authority records, similar to
previous sessions for catalog records. See document with screenshot attached.

-- 
You are receiving this mail because:
You are watching all bug changes.