[Bug 7594] New: Google Cover CSS suggestions
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7594 Bug #: 7594 Summary: Google Cover CSS suggestions Classification: Unclassified Change sponsored?: --- Product: Koha Version: rel_3_6 Platform: All OS/Version: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC AssignedTo: oleonard@myacpl.org ReportedBy: ssammons@npelem.com QAContact: koha.sekjal@gmail.com In google-jackets.js there is hard coded "style" on line 48 using... http://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=koha-tmpl/opac-tmp... I suggest this is moved to a .css file, as to allow the end user to impose changes to the style. Additionally, it would be nice if a unique class can be added to all the image preview types. Currently they are all dynamic (class & id). Added a class such as "gbs-cover" or similar would allow end users to use CSS to uniquely style each book cover used. I am not able to find where this is generated or I would suggest the lines of code to change. Finally, the "No Image Available" does not appear in the HTML source code. The JS shows it creating a "span" element with a class of no-image. Is there anyway to actually have that appear in the actual source code? All suggested changes should allow for an end user to have easier and clearer control of OPAC search generated images. -- 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=7594 Liz Rea <wizzyrea@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |wizzyrea@gmail.com --- Comment #1 from Liz Rea <wizzyrea@gmail.com> 2012-02-24 19:54:31 UTC --- (In reply to comment #0)
In google-jackets.js there is hard coded "style" on line 48 using... http://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=koha-tmpl/opac-tmp...
I suggest this is moved to a .css file, as to allow the end user to impose changes to the style.
Changes that make the styling of the OPAC more granular are always welcome.
Additionally, it would be nice if a unique class can be added to all the image preview types. Currently they are all dynamic (class & id). Added a class such as "gbs-cover" or similar would allow end users to use CSS to uniquely style each book cover used. I am not able to find where this is generated or I would suggest the lines of code to change.
A good point.
Finally, the "No Image Available" does not appear in the HTML source code. The JS shows it creating a "span" element with a class of no-image. Is there anyway to actually have that appear in the actual source code?
All suggested changes should allow for an end user to have easier and clearer control of OPAC search generated images.
-- 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=7594 --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- (In reply to comment #0)
In google-jackets.js there is hard coded "style" on line 48 using... http://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=koha-tmpl/opac- tmpl/prog/en/js/google-jackets.js;h=70b1b84337c4c3807cd8e1997a2fda6cd9563004; hb=HEAD
The line in question reads: $("."+book.bib_key).each(function() { This is the script's way of looping over elements with that particular class (i.e. any title with that same ISBN.). There is nothing at that line which sets a hard-coded style. There is an invalid style hard-coded in opac-results.tt at line 542 and 543 (in master): <div style="block"... The correct way to say that would be <div style="display:block"... but that is redundant for divs, so I recommend we remove the style attribute.
Finally, the "No Image Available" does not appear in the HTML source code.
It does in two places. In the template (opac-results.tt) it is output if the record has no ISBN: [% IF ( SEARCH_RESULT.normalized_isbn ) %]...[% ELSE %] <span class="no-image">No cover image available</span>[% END %] Then the various scripts output the same thing if they don't return an image. For instance, in google-jackets.js: var message = document.createElement("span"); $(message).attr("class","no-image"); $(message).html(NO_GOOGLE_JACKET); $(this).append(message); This creates the <span> element, adds the "no-image" class, fills the tag with the NO_GOOGLE_JACKET message (specified in opac-result.tt) and appends the newly-built <span> to the <div>. The "no-image" class can be modified globally, but if you want multiple cover sources enabled at the same time and want the messages styled differently you'll have to add another class as well, both in the JS and the template: $(message).attr("class","no-image google"); Then your custom stylesheet could address it: span.no-image.google { border: 3px dotted blue; } And you could write some JS for OpacUserJS to change the message using that class to target it. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7594 --- Comment #3 from Shane Sammons <ssammons@npelem.com> --- Yeah, we addressed everything today via IRC... However, I made a mistake, not line 48, but 58 has the following: $(this).append( 58 '<div style="margin-bottom:5px; margin-top:-5px;font-size:9px">' + 59 '<a '+target+'href="' + 60 book.info_url + 61 '"><img src="' + 62 'http://books.google.com/intl/en/googlebooks/images/gbs_preview_sticker1.gif' + 63 '"></a></div>' We should probably fix that to be a class and put it in the .css file. I will look into what file (hopefully tomorrow). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7594 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7594 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Google Cover CSS |Google Cover Javascript |suggestions |contains hardcoded CSS | |style Version|3.6 |master --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- On current master: http://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=koha-tmpl/opac-tmp... -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org