Bugzilla – Attachment 70783 Details for
Bug 20051
Invalid markup in staff client's header.inc
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 20051: Invalid markup in staff client's header.inc
Bug-20051-Invalid-markup-in-staff-clients-headerin.patch (text/plain), 3.53 KB, created by
Charles Farmer
on 2018-01-19 21:59:39 UTC
(
hide
)
Description:
Bug 20051: Invalid markup in staff client's header.inc
Filename:
MIME Type:
Creator:
Charles Farmer
Created:
2018-01-19 21:59:39 UTC
Size:
3.53 KB
patch
obsolete
>From 681bc32945d8f0550a6aef60be38dff4890c937c Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Fri, 19 Jan 2018 19:41:35 +0000 >Subject: [PATCH] Bug 20051: Invalid markup in staff client's header.inc > >This patch corrects markup errors in header.inc which introduced by >Bug 19851. For id attributes on <span>s have been replaced with classes, >eliminating an HTML validator error about duplicate ids. > >Minor unrelated change: Updated an element's id with a more descriptive >name. > >Test plan the same as Bug 19851's: > > - Log in to the staff client and confirm that the appearance of the > menu bar at the top of the screen looks correct at various browser > widths. > - Confirm that the logged-in user menu label and help link expand and > collapse according to browser width. > >Signed-off-by: Charles Farmer <charles.farmer@inLibro.com> >--- > koha-tmpl/intranet-tmpl/prog/en/includes/header.inc | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc >index 4f3d579..87dbe70 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc >+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc >@@ -50,7 +50,7 @@ > <ul class="nav navbar-nav navbar-right"> > [% IF ( loggedinusername ) %] > <li class="dropdown"> >- <a href="#" id="drop3" role="button" class="dropdown-toggle" data-toggle="dropdown"> >+ <a href="#" id="logged-in-menu" role="button" class="dropdown-toggle" data-toggle="dropdown"> > <span id="logged-in-info-brief"> > <i class="fa fa-user"></i> > </span> >@@ -66,15 +66,15 @@ > [% IF ( LoginBranchname == 'NO_LIBRARY_SET' ) %] > NO LIBRARY SET > [% ELSE %] >- <span id="logged-in-branch-name">[% LoginBranchname %]</span> >- <span id="logged-in-branch-code" class="content_hidden">[% Branches.GetLoggedInBranchcode %]</span> >+ <span class="logged-in-branch-name">[% LoginBranchname %]</span> >+ <span class="logged-in-branch-code content_hidden">[% Branches.GetLoggedInBranchcode %]</span> > [% END %] > </strong> > [% END %] > </span> > <b class="caret"></b> > </a> >- <ul class="dropdown-menu" role="menu" aria-labelledby="drop3"> >+ <ul class="dropdown-menu" role="menu" aria-labelledby="logged-in-menu"> > > <li class="loggedin-menu-label"> > Logged in as:<br /> >@@ -90,8 +90,8 @@ > NO LIBRARY SET > [% ELSE %] > Location: <br /> >- <span id="logged-in-branch-name">[% LoginBranchname %]</span> >- <span id="logged-in-branch-code" class="content_hidden">[% Branches.GetLoggedInBranchcode %]</span> >+ <span class="logged-in-branch-name">[% LoginBranchname %]</span> >+ <span class="logged-in-branch-code content_hidden">[% Branches.GetLoggedInBranchcode %]</span> > [% END %] > [% END %] > </li> >-- >2.7.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 20051
:
70777
|
70783
|
70973
|
71086