[Bug 23903] New: Replace OPAC icons with Font Awesome
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Bug ID: 23903 Summary: Replace OPAC icons with Font Awesome Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard@myacpl.org Reporter: oleonard@myacpl.org QA Contact: testopia@bugs.koha-community.org We can eliminate use of the sprite image for icons and use Font Awesome instead. This will be simpler and more customization-friendly. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- I still need to write up a proper test plan and commit message, but if anyone is interested in looking: https://gitlab.com/koha-dev/koha-dev/commits/ip-opac-fa-icons -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff Patch complexity|--- |Medium patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- The remote branch has been updated with more fixes and a complete test plan. I've also posted patches for download (they're too large to attach here): https://zivotdesign.com/download/bug-23903-1.patch https://zivotdesign.com/download/bug-23903-2.patch Test plan: Bug 23903: Replace OPAC icons with Font Awesome This patch replaces CSS-based background image icons with Font Awesome icons. This will eliminate the use of sprite.png altogether. To test, apply the patch and regenerate the OPAC CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff...). Because JavaScript events are often triggered by class, the changes made in this patch require testing of JavaScript-driven behavior as well as visual confirmation of the changes. Enable OPAC features to expose them for testing: Login, Holds, Article requests, Tags, Cart, Search term highlighting. - Perform a search in the OPAC - In the header at the top of the search results, check the 'Unhighlight/Highlight' link. - Select some search results and verify that the multi-item controls are enabled: Add to cart/Lists; Place hold; Tag. Check that these controls work correctly. - Verify that the following links look correct and work correctly: 'Place hold,' 'Request article,' 'Log in to add tags,' 'Add to cart.' - Log in to the OPAC and return to the search results page. Check the 'Add tag' and 'Save to lists' links. - View the lists page - Test the 'Edit' and 'Delete' controls. - View the contents of a list. Test the following: - Controls at the top of the table of results ('New list,' 'Download list,' 'Send list,' 'Print list,' 'Edit list,' and 'Delete list'). - Multi-selection operations: 'Place hold,' 'Tag,' and 'Remove from list.' - Controls for each item on the list: 'Place hold,' 'Add tag,' 'Save to another list,' 'Remove from this list,' and 'Add to cart.' - View the bibliographic detail page for any search result - In the right-hand sidebar menu, verify that these controls look correct and work correctly: 'Place hold,' 'Print,' 'Request article,' 'Save to your lists,' 'Add to your cart,' 'Unhighlight,' 'Send to device,' 'Save record,' and 'More searches.' - Test the shelf browser by clicking 'Browse shelf' under the call number for any of the holdings. - Test the 'Next' and 'Previous' controls. - Check the tabs for switching between 'Normal,' 'MARC,' and 'ISBD' views and verify that they look correct on each of those pages. - Add some items to the Cart and open the Cart - Test the cart controls at the top: 'More details/Brief display,' 'Send,' 'Download,' 'Empty and close,' 'Hide window,' 'Print.' - Test the multiple-selection controls: 'Remove,' 'Add to list,' 'Place hold,' and 'Tag.' - Log in to the OPAC as a user who has items checked out which can be renewed. On the 'Your summary' page, on the 'Checked out' tab, test the 'Renew selected' and 'Renew all' links. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |23913 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23913 [Bug 23913] Use a single menu to sort lists in the OPAC -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Liz Rea <wizzyrea@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |wizzyrea@gmail.com --- Comment #3 from Liz Rea <wizzyrea@gmail.com> --- I went through your plan, all of the icons were looking so lovely and modern. All the yes to this patch, didn't spot anything amiss. You may have my sign off! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |martin.renvoize@ptfs-europe | |.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |rel_19_11_candidate -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #4 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- This is a great change.. passing QA and pushing. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |19.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 --- Comment #5 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Nice work! Pushed to master for 19.11.00 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |martin.renvoize@ptfs-europe |y.org |.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|rel_19_11_candidate | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This patchset updates the release notes| |majority of icons in the | |opac to use fontawesome | |icons. It improves the | |consistency of icons, icon | |alignment and also gives a | |minor performance boost in | |opac display. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 --- Comment #6 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 94998 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=94998&action=edit Bug 23903: fix selenium tests -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de Text to go in the|This patchset updates the |This patchset updates the release notes|majority of icons in the |majority of icons in the |opac to use fontawesome |opac to use Font Awesome |icons. It improves the |icons. It improves the |consistency of icons, icon |consistency of icons, icon |alignment and also gives a |alignment and also gives a |minor performance boost in |minor performance boost in |opac display. |opac display. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23903 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=24689 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org