[Koha-bugs] [Bug 6902] Add integration for EBSCO's Novelist Select service
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Fri Oct 7 17:28:15 CEST 2011
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6902
Owen Leonard <oleonard at myacpl.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #5548|0 |1
is obsolete| |
--- Comment #4 from Owen Leonard <oleonard at myacpl.org> 2011-10-07 15:28:15 UTC ---
Created attachment 5777
--> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=5777
Signed-off patch
These patches tested fine for me. The changes to the code are minimal, since
the Novelist data, like LibraryThing data, is pulled in via JavaScript. A
couple of comments, neither of which would prevent this from going forward:
The option to put the content in the right-hand sidebar doesn't really work
well because of the quantity of content. It's too squeezed. Since the position
of the content is controlled by a system preference, this is easily avoided.
A good follow-up might be to add <noscript> text following the model of the
LibraryThing content. Otherwise users with JS disabled will see the Novelist
heading but no content and no explanation.
--
Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
More information about the Koha-bugs
mailing list