Bug 10566 - Improve OPAC course reserves pages
Summary: Improve OPAC course reserves pages
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Owen Leonard
QA Contact:
URL:
Keywords:
Depends on:
Blocks: 10632
  Show dependency treegraph
 
Reported: 2013-07-10 13:50 UTC by Owen Leonard
Modified: 2014-12-07 20:02 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 10566 - Improve OPAC course reserves pages (9.71 KB, patch)
2013-07-10 13:58 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 10566 - Improve OPAC course reserves pages (9.72 KB, patch)
2013-07-16 09:27 UTC, Kyle M Hall
Details | Diff | Splinter Review
[PASSED QA] Bug 10566 - Improve OPAC course reserves pages (9.91 KB, patch)
2013-08-25 19:51 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2013-07-10 13:50:40 UTC
The new course reserves pages could use some improvements: More specific titles, improved labeling, other markup enhancements.
Comment 1 Owen Leonard 2013-07-10 13:58:54 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall 2013-07-16 09:27:09 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2013-08-25 19:51:44 UTC
Created attachment 20606 [details] [review]
[PASSED QA] Bug 10566 - Improve OPAC course reserves pages

This patch makes various improvements to the OPAC course reserve pages:

- Descriptive page titles for both pages.
- More compact display of course information on course detail page
  (a list instead of a table, consistent with other displays of similar
  information in Koha).
- Correct capitalization.
- Added label and fieldset to course reserve search form.
- Added breadcrumb navigatio nto course reserve detail page as a path
  back to course reserves main page.

To test, apply the patch and confirm that the changes haven't broken
anything and are an improvement.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes all tests and improves the display.
Tested with only 1 course, 2 courses,
courses with and without items selected.
Comment 4 Galen Charlton 2013-08-28 15:08:23 UTC
Pushed to master, along with a follow-up to fix a couple stray HTML validation issues.  Thanks, Owen!