There are a few inconsistencies in the new CSS for 3.8 - namely: - missing gradient class on returns.tt - div.action had a white background instead of blue - gradient in header had incorrect color choice (purple) for chrome browsers And a few small improvements: - adds slight transparency for news so logo shows through on mainpage.. - reverse colors of menu div - blue for inactive, grey for active. And probably more that I haven't found yet.
Created attachment 9279 [details] [review] Bug 7998 - CSS fixes for new interface adds slight transparency for news so logo shows through on mainpage.. Fixes purple header gradient in Chrome-based browsers. remove list from returns.tt options so checkboxes do not have bullets. fix missing gradient
An additional problem has occurred with the top nav bar. Here is part of the discussion from irc: chris_n> but things do bounce around a bit when I click log out <oleonard> ? <chris_n> the cart count shows up as (0) for a moment after the link is clicked <chris_n> in chrome I should add <oleonard> That's the automatic carty-emptying routine <chris_n> so probably a css fix? <oleonard> When you say bounce around you mean the toolbar gets wider for a moment? <chris_n> yes, vertically <oleonard> The bigger problems is that it happens whenever there's anything in the cart <oleonard> The new design removed the explicit width on the div which contains that menu <oleonard> When the cart updates the container, for whatever reason, doesn't expand when the text changes <oleonard> #toplevelnav { width: 40%; } might be a fix <chris_n> fwiw, that fixes it here <oleonard> One concern is longer-character words in translations <chris_n> so the problem would show up when I had items in my cart? <oleonard> Yes <chris_n> ouch... so it is a bigger problem
Created attachment 9291 [details] [review] Bug 7998 - UI tweaks for 3.8 turns searchheader blue, rounds corners, improves spacing for sort form.
also 2nd patch is on top of the first.
Created attachment 9300 [details] [review] Bug 7998 - CSS fixes for new interface adds slight transparency for news so logo shows through on mainpage.. Fixes purple header gradient in Chrome-based browsers. remove list from returns.tt options so checkboxes do not have bullets. fix missing gradient class on returns.tt. reverse colors of menu div - blue for inactive, grey for active. turns searchheader blue, rounds corners, improves spacing for sort form. Adds padding, rounded corners, and a 1px border to the now-blue toolbar. increase width of intranet nav div to width 40% add a bit of padding to #searchheader fieldset.action changes - removed background, added a little padding to make it look better in all of the uses I could find of it.
Created attachment 9301 [details] [review] Bug 7998 - Followup - round corners of search input box
Created attachment 9302 [details] [review] [SIGNED-OFF] Bug 7998 - CSS fixes for new interface adds slight transparency for news so logo shows through on mainpage.. Fixes purple header gradient in Chrome-based browsers. remove list from returns.tt options so checkboxes do not have bullets. fix missing gradient class on returns.tt. reverse colors of menu div - blue for inactive, grey for active. turns searchheader blue, rounds corners, improves spacing for sort form. Adds padding, rounded corners, and a 1px border to the now-blue toolbar. increase width of intranet nav div to width 40% add a bit of padding to #searchheader fieldset.action changes - removed background, added a little padding to make it look better in all of the uses I could find of it. Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
Created attachment 9303 [details] [review] [SIGNED-OFF] Bug 7998 - Followup - round corners of search input box Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
This corrects the 'Cart' button issue mentioned and makes other desirable cosmetic changes as well. Nice work!
Created attachment 9304 [details] [review] Bug 7998 - followup - make facets header background blue
Created attachment 9305 [details] [review] [SIGNED_OFF] Bug 7998 - CSS fixes for new interface A squash of the following three commits... adds slight transparency for news so logo shows through on mainpage.. Fixes purple header gradient in Chrome-based browsers. remove list from returns.tt options so checkboxes do not have bullets. fix missing gradient class on returns.tt. reverse colors of menu div - blue for inactive, grey for active. turns searchheader blue, rounds corners, improves spacing for sort form. Adds padding, rounded corners, and a 1px border to the now-blue toolbar. increase width of intranet nav div to width 40% add a bit of padding to #searchheader fieldset.action changes - removed background, added a little padding to make it look better in all of the uses I could find of it. Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu> Bug 7998 - Followup - round corners of search input box Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu> Bug 7998 - followup - make facets header background blue Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
Template and CSS only changes. Template change involves replacing a <ul> and <li> elements with corresponding <p>; done safely and in an XHTML compatible way. I trust that the layout and coloration changes look good in testing, so marking as Passed QA.
Created attachment 9371 [details] [review] Bug 7998 - followup - make facets header background blue bug 7998 - followup - fixing headers on search.pl to be blue, rounded. bug 7998 - followup - consistency tweaks match menu borders to the search header tabs (green border) hover menus a very light grey instead of #eee. make fieldset.brief have a consistent border with the rest of the fieldsets. bug 7998 - followup - more tabs/borders updating to fit in new look boraccount.pl bug 7998 - followup - add gradient div to prefs-admin-search.inc
2nd patch here is a followp, will set back to needs signoff when/if this is into master.
*** Bug 8047 has been marked as a duplicate of this bug. ***
I don't know if this page is included in all of this but on cgi-bin/koha/admin/fieldmapping.pl at the bottom there is a white bar where the submit button is when it should be blue.
another one: /cgi-bin/koha/labels/spinelabel-home.pl
I'm not seeing a problem with either of the pages mentioned above. Both pages display the submit, etc. button properly afaict. Kohaversion: 3.08.00.000
I'm OK to push this patch, but also adding Gaetan to this bug, for having him checking i'm not missing something. If I have a question, it could be = why adding round corners to search toolbar and not to every input field in the interface ? Gaetan, I'll push once you've confirmed there is nothing wrong & hidden with those patches.
Nicole - those issues are addressed in the first patch.
(In reply to comment #20) > Nicole - those issues are addressed in the first patch. That explains why I did not see them. :-)
Is attachment 9371 [details] [review] a follow-up to the signed-off patch? There are problems I see with the signed-off patch which I would like to see resolved. I'm wondering if the second does it but it doesn't apply for me. Applying: Bug 7998 - followup - make facets header background blue fatal: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge.
Created attachment 9458 [details] [review] Bug 7998 - CSS fixes for new interface adds slight transparency for news so logo shows through on mainpage.. Fixes purple header gradient in Chrome-based browsers. remove list from returns.tt options so checkboxes do not have bullets. fix missing gradient class on returns.tt. reverse colors of menu div - blue for inactive, grey for active. turns searchheader blue, rounds corners, improves spacing for sort form. Adds padding, rounded corners, and a 1px border to the now-blue toolbar. increase width of intranet nav div to width 40% add a bit of padding to #searchheader fieldset.action changes - removed background, added a little padding to make it look better in all of the uses I could find of it. Bug 7998 - Followup - round corners of search input box Bug 7998 - followup - make facets header background blue bug 7998 - followup - fixing headers on search.pl to be blue, rounded. bug 7998 - followup - consistency tweaks match menu borders to the search header tabs (green border) hover menus a very light grey instead of #eee. make fieldset.brief have a consistent border with the rest of the fieldsets. bug 7998 - followup - more tabs/borders updating to fit in new look boraccount.pl bug 7998 - followup - add gradient div to prefs-admin-search.inc
Created attachment 9459 [details] Screenshot showing the toolbar colors added by this patch. I disagree with the change which adds a border-radius to the header search input fields. That change does not work well in all browsers. In Firefox, for instance, the bottom and right borders disappear completely. I would like to get more opinions about the toolbar background and border colors. To me this change gives the toolbar a little too much weight. I'd rather let the buttons be the focus.
Created attachment 9460 [details] Screenshot showing the toolbar colors before this patch. This is how it would look if no changes were made to the toolbar.
Created attachment 9461 [details] Screenshot showing the toolbar with different colors. I lean towards something like this: No border and a lighter background. I would love to hear others' opinions.
Comment on attachment 9461 [details] Screenshot showing the toolbar with different colors. I like the lighter background without border best of the 3 screenshots.
I like the first screenshot. I think it defines the toolbar better as a separate entity on the page as well as breaking up the page a bit.
I can get behind the lighter color on the toolbar (I was just grabbing existent colors in Gaetan's scheme), however I like the border and the corners (and I'm willing to fix it so that it looks nice in as many browsers as we can make it look nice in).
dear friends, how about this? http://screencast.com/t/5ijg46J9aC Lighter color, barely discernible border.
I really like all this and certainly don't oppose anything :). Actually there is a number of other pages where i noticed the gradient is missing, all in the admin : admin/preferences.pl, admin/categorie.pl, admin/cities.pl, admin/roadtype.pl, admin/currency.pl, admin/aqbudgetperiods.pl, /admin/aqbudgets.pl, admin/z3950servers.pl. Maybe i missed other ones Also there are some pages on which the header search doesn't show at all. I think this should be standardised so that it shows everywhere. authorities/authorities.pl cataloguing/additem.pl cataloguing/addbiblio.pl search.pl (having the header search for check out and check in here would still make sense)
(In reply to comment #31) > Also there are some pages on which the header search doesn't show at all. I > think this should be standardised so that it shows everywhere. > authorities/authorities.pl > cataloguing/additem.pl > cataloguing/addbiblio.pl The header search was left off these pages deliberately because of the specialized nature of the interfaces. The assumption is that these pages are part of a process which must be completed, so extra focus is given to the page. > search.pl (having the header search for check out and check in here would > still make sense) The default header search was left off this page to eliminate ambiguity with regard to catalog search boxes, but adding a search with just check out and check in makes sense to me.
Created attachment 9462 [details] [review] Bug 7998 - Change toolbar to be lighter, with barely discernible border Will need to be applied after the other patch.
(In reply to comment #31) > I really like all this and certainly don't oppose anything :). > > Actually there is a number of other pages where i noticed the gradient is > missing, all in the admin : admin/preferences.pl, admin/categorie.pl, > admin/cities.pl, admin/roadtype.pl, admin/currency.pl, > admin/aqbudgetperiods.pl, /admin/aqbudgets.pl, admin/z3950servers.pl. Maybe > i missed other ones I'm happy to look after getting these done where it makes sense to do so. :)
Created attachment 9463 [details] [review] Bug 7998 - CSS fixes for new interface adds slight transparency for news so logo shows through on mainpage.. Fixes purple header gradient in Chrome-based browsers. remove list from returns.tt options so checkboxes do not have bullets. fix missing gradient class on returns.tt. reverse colors of menu div - blue for inactive, grey for active. turns searchheader blue, rounds corners, improves spacing for sort form. Adds padding, rounded corners, and a 1px border to the now-blue toolbar. increase width of intranet nav div to width 40% add a bit of padding to #searchheader fieldset.action changes - removed background, added a little padding to make it look better in all of the uses I could find of it. Bug 7998 - Followup - round corners of search input box Bug 7998 - followup - make facets header background blue bug 7998 - followup - fixing headers on search.pl to be blue, rounded. bug 7998 - followup - consistency tweaks match menu borders to the search header tabs (green border) hover menus a very light grey instead of #eee. make fieldset.brief have a consistent border with the rest of the fieldsets. bug 7998 - followup - more tabs/borders updating to fit in new look boraccount.pl bug 7998 - followup - add gradient div to prefs-admin-search.inc Bug 7998 - Change toolbar to be lighter, with barely discernible border Will need to be applied after the other patch. Bug 7998 - add gradient to roadtype admin panel Bug 7998 - adds gradient to patrons-admin-search.tt Bug 7998 - add gradient to budgets-admin-search.inc bug 7998 - add gradient to z3950-admin-search.inc Bug 7998 - add gradient to cities-admin-search.inc
Any chance we can get what's been done pushed on up to fix the broken-ness?
Created attachment 9464 [details] [review] Bug 7998 - CSS fixes for new interface Omnibus of changes thus far: adds slight transparency for news so logo shows through on mainpage.. Fixes purple header gradient in Chrome-based browsers. remove list from returns.tt options so checkboxes do not have bullets. fix missing gradient class on returns.tt. reverse colors of menu div - blue for inactive, grey for active. turns searchheader blue, rounds corners, improves spacing for sort form. Adds padding, rounded corners, and a 1px border to the now-blue toolbar. increase width of intranet nav div to width 40% add a bit of padding to #searchheader fieldset.action changes - removed background, added a little padding to make it look better in all of the uses I could find of it. Bug 7998 - Followup - round corners of search input box Bug 7998 - followup - make facets header background blue bug 7998 - followup - fixing headers on search.pl to be blue, rounded. bug 7998 - followup - consistency tweaks match menu borders to the search header tabs (green border) hover menus a very light grey instead of #eee. make fieldset.brief have a consistent border with the rest of the fieldsets. bug 7998 - followup - more tabs/borders updating to fit in new look boraccount.pl bug 7998 - followup - add gradient div to prefs-admin-search.inc Bug 7998 - Change toolbar to be lighter, with barely discernible border Will need to be applied after the other patch. Bug 7998 - add gradient to roadtype admin panel Bug 7998 - adds gradient to patrons-admin-search.tt Bug 7998 - add gradient to budgets-admin-search.inc bug 7998 - add gradient to z3950-admin-search.inc Bug 7998 - add gradient to cities-admin-search.inc
bother I forgot to take off the rounded corners per Owen's request.
Created attachment 9465 [details] [review] Bug 7998 - CSS fixes for new interface Omnibus of changes thus far: adds slight transparency for news so logo shows through on mainpage.. Fixes purple header gradient in Chrome-based browsers. remove list from returns.tt options so checkboxes do not have bullets. fix missing gradient class on returns.tt. reverse colors of menu div - blue for inactive, grey for active. turns searchheader blue, rounds corners, improves spacing for sort form. Adds padding, rounded corners, and a 1px border to the now-blue toolbar. increase width of intranet nav div to width 40% add a bit of padding to #searchheader fieldset.action changes - removed background, added a little padding to make it look better in all of the uses I could find of it. Bug 7998 - followup - make facets header background blue bug 7998 - followup - fixing headers on search.pl to be blue, rounded. bug 7998 - followup - consistency tweaks match menu borders to the search header tabs (green border) hover menus a very light grey instead of #eee. make fieldset.brief have a consistent border with the rest of the fieldsets. bug 7998 - followup - more tabs/borders updating to fit in new look boraccount.pl bug 7998 - followup - add gradient div to prefs-admin-search.inc Bug 7998 - Change toolbar to be lighter, with barely discernible border Will need to be applied after the other patch. Bug 7998 - add gradient to roadtype admin panel Bug 7998 - adds gradient to patrons-admin-search.tt Bug 7998 - add gradient to budgets-admin-search.inc bug 7998 - add gradient to z3950-admin-search.inc Bug 7998 - add gradient to cities-admin-search.inc bug 7998 - active tab on checkout table now has green border like side menu
Created attachment 9532 [details] [review] Bug 7998 - CSS fixes for new interface Omnibus of changes thus far: adds slight transparency for news so logo shows through on mainpage.. Fixes purple header gradient in Chrome-based browsers. remove list from returns.tt options so checkboxes do not have bullets. fix missing gradient class on returns.tt. reverse colors of menu div - blue for inactive, grey for active. turns searchheader blue, rounds corners, improves spacing for sort form. Adds padding, rounded corners, and a 1px border to the now-blue toolbar. increase width of intranet nav div to width 40% add a bit of padding to #searchheader fieldset.action changes - removed background, added a little padding to make it look better in all of the uses I could find of it. Bug 7998 - followup - make facets header background blue bug 7998 - followup - fixing headers on search.pl to be blue, rounded. bug 7998 - followup - consistency tweaks match menu borders to the search header tabs (green border) hover menus a very light grey instead of #eee. make fieldset.brief have a consistent border with the rest of the fieldsets. bug 7998 - followup - more tabs/borders updating to fit in new look boraccount.pl bug 7998 - followup - add gradient div to prefs-admin-search.inc Bug 7998 - Change toolbar to be lighter, with barely discernible border Will need to be applied after the other patch. Bug 7998 - add gradient to roadtype admin panel Bug 7998 - adds gradient to patrons-admin-search.tt Bug 7998 - add gradient to budgets-admin-search.inc bug 7998 - add gradient to z3950-admin-search.inc Bug 7998 - add gradient to cities-admin-search.inc bug 7998 - active tab on checkout table now has green border like side menu Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Created attachment 9537 [details] [review] Bug 7998: Don't make the news all bold
Looks great and takes care of a lot of thing that needed to be fixed. Signed off and approved!
QA comment : mostly CSS changes, passed QA
patch pushed, don't forget to fully reload your browser if you want to test, the old CSS is in cache !
This fix will be in 3.8.1