Bug 11800 - Correct toolbar markup on course reserve pages
Summary: Correct toolbar markup on course reserve pages
Status: RESOLVED DUPLICATE of bug 11713
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-20 14:59 UTC by Owen Leonard
Modified: 2014-03-17 17:06 UTC (History)
1 user (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 11800 - Correct toolbar markup on course reserve pages (5.92 KB, patch)
2014-02-20 15:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 11800 - Correct toolbar markup on course reserve pages (5.93 KB, patch)
2014-02-28 18:16 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2014-02-20 14:59:37 UTC
The toolbar markup on a couple of the course reserve pages doesn't exactly match other instances in Koha.
Comment 1 Owen Leonard 2014-02-20 15:03:18 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall 2014-02-28 18:16:25 UTC
Created attachment 25738 [details] [review]
Bug 11800 - Correct toolbar markup on course reserve pages

This patch corrects some inconsistencies between the toolbars on some
course reserves pages and other toolbars in Koha.

Other changes: Removed obsolete commented markup and corrected some
invalid markup.

To test course reserves must be enabled. View the list of course
reserves (course_reserves/course-reserves.pl) and the detail view of a
course (course_reserves/course-details.pl?course_id=X). Toolbars on both
pages should look correct and work correctly.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 3 Katrin Fischer 2014-03-16 13:56:46 UTC
There are merge conflicts in 2 of the tt files - I don't want to make a mistake here. Owen, can you please rebase and switch back to Signed off?
Comment 4 Owen Leonard 2014-03-17 17:06:14 UTC

*** This bug has been marked as a duplicate of bug 11713 ***