[Bug 7530] New: Missing <li> tag in circ-menu.inc
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7530 Bug #: 7530 Summary: Missing <li> tag in circ-menu.inc Classification: Unclassified Change sponsored?: --- Product: Koha Version: unspecified Platform: All OS/Version: All Status: NEW Severity: minor Priority: P5 - low Component: Templates AssignedTo: oleonard@myacpl.org ReportedBy: gcollum@gmail.com QAContact: koha.sekjal@gmail.com The address fields of the patron information display of circ-menu.inc is not within <li> tags creating invalid XHTML on several pages. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7530 Garry Collum <gcollum@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff AssignedTo|oleonard@myacpl.org |gcollum@gmail.com --- Comment #1 from Garry Collum <gcollum@gmail.com> 2012-02-11 17:47:07 UTC --- Created attachment 7576 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=7576 Proposed patch Trivial patch that corrects an XHTML problem on several borrower pages. Can be tested by going to borrower checkout and validating the XHTML for the borrower information side menu. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7530 Liz Rea <wizzyrea@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #7576|0 |1 is obsolete| | --- Comment #2 from Liz Rea <wizzyrea@gmail.com> 2012-02-14 19:22:31 UTC --- Created attachment 7645 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=7645 [Signed Off] Bug 7530: Missing <li> tag in circ-menu.inc The address information of circ-menu.inc is not contained in <li> tags, this patch fixes it. Signed-off-by: Liz Rea <wizzyrea@gmail.com> verified new <li> in template. Looks good. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7530 Liz Rea <wizzyrea@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |wizzyrea@gmail.com -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7530 Ian Walls <koha.sekjal@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #3 from Ian Walls <koha.sekjal@gmail.com> 2012-02-16 16:07:40 UTC --- Adds missing start element to list. This seems to have already been fixed in the .tt version of this file, so marking this simple template change as Passed QA. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7530 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |paul.poulain@biblibre.com Version|unspecified |rel_3_6 --- Comment #4 from Paul Poulain <paul.poulain@biblibre.com> 2012-02-17 09:12:09 UTC --- This patch add only a <li>, the </li> was already here, so the HTML is now more valid than before the patch -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7530 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable --- Comment #5 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This fix was included in the 3.6.x branch prior to 3.6.4. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org