As part of Bug 32067 - [OMNIBUS] Consistent classes for submit and/or primary buttons: This patch will update buttons in catalog templates so that buttons which are currently styled as primary buttons will have the the "btn btn-primary" classes. In some places, the classes have been added to buttons which were not input[type='submit'] but which should be styled as primary.
Created attachment 142950 [details] [review] Bug 32071: Consistent classes for primary buttons: Catalog This patch makes changes the button markup in Catalog templates -- including the Cart -- so that all submit buttons and any buttons that should should be styled as primary buttons have the Bootstrap class "btn btn-primary." To test, apply the patch and view pages the catalog to confirm that everything looks correct. In most cases there are no visible changes.
Created attachment 143300 [details] 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.
Created attachment 143518 [details] [review] Bug 32071: Consistent classes for primary buttons: Catalog This patch makes changes the button markup in Catalog templates -- including the Cart -- so that all submit buttons and any buttons that should should be styled as primary buttons have the Bootstrap class "btn btn-primary." Edit: This revised patch changes the button styles on the search history page to make them more consistent with buttons in similar interfaces (see catalog search results). The patch also adds Bootstrap size classes to the item detail page and removes some global CSS which targeted buttons on that page. To test, apply the patch and view pages the catalog to confirm that everything looks correct. In most cases there are no visible changes. - Advanced search: The main search button at the top - Search results: The "Search within results" submit button - Bibliographic detail page: This change is to a hidden button. This section can be removed in a separate bug. - Bibliographic detail page -> Items: Various "Update" and "Set status" buttons. Note that the alignment of buttons has not changed in the redesign. - Item search: The main search button at the top - Search history (from the logged-in user's menu in the top right): "Delete" buttons in each section - Catalog -- Current and previous sessions - Authority -- Current and previous sessions - In the Cart popup window, click "Send." In the resulting popup, the "Send button. - The template basket/downloadcart.tt has been modified but I think the template might be unused.
Created attachment 143630 [details] [review] Bug 32071: Consistent classes for primary buttons: Catalog This patch makes changes the button markup in Catalog templates -- including the Cart -- so that all submit buttons and any buttons that should should be styled as primary buttons have the Bootstrap class "btn btn-primary." Edit: This revised patch changes the button styles on the search history page to make them more consistent with buttons in similar interfaces (see catalog search results). The patch also adds Bootstrap size classes to the item detail page and removes some global CSS which targeted buttons on that page. To test, apply the patch and view pages the catalog to confirm that everything looks correct. In most cases there are no visible changes. - Advanced search: The main search button at the top - Search results: The "Search within results" submit button - Bibliographic detail page: This change is to a hidden button. This section can be removed in a separate bug. - Bibliographic detail page -> Items: Various "Update" and "Set status" buttons. Note that the alignment of buttons has not changed in the redesign. - Item search: The main search button at the top - Search history (from the logged-in user's menu in the top right): "Delete" buttons in each section - Catalog -- Current and previous sessions - Authority -- Current and previous sessions - In the Cart popup window, click "Send." In the resulting popup, the "Send button. - The template basket/downloadcart.tt has been modified but I think the template might be unused. Signed-off-by: David Nind <david@davidnind.com>
Created attachment 143632 [details] [review] Bug 32071: Consistent classes for primary buttons: Catalog This patch makes changes the button markup in Catalog templates -- including the Cart -- so that all submit buttons and any buttons that should should be styled as primary buttons have the Bootstrap class "btn btn-primary." Edit: This revised patch changes the button styles on the search history page to make them more consistent with buttons in similar interfaces (see catalog search results). The patch also adds Bootstrap size classes to the item detail page and removes some global CSS which targeted buttons on that page. To test, apply the patch and view pages the catalog to confirm that everything looks correct. In most cases there are no visible changes. - Advanced search: The main search button at the top - Search results: The "Search within results" submit button - Bibliographic detail page: This change is to a hidden button. This section can be removed in a separate bug. - Bibliographic detail page -> Items: Various "Update" and "Set status" buttons. Note that the alignment of buttons has not changed in the redesign. - Item search: The main search button at the top - Search history (from the logged-in user's menu in the top right): "Delete" buttons in each section - Catalog -- Current and previous sessions - Authority -- Current and previous sessions - In the Cart popup window, click "Send." In the resulting popup, the "Send button. - The template basket/downloadcart.tt has been modified but I think the template might be unused. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 22.11. Nice work everyone, thanks!