Created attachment 60002 [details] [review] Bug 18078: Updating buttons for z39.50 Authority search 1) Go to Cataloguing -> New from Z39.50 2) Notice how Search button is strangely off page (have to scroll down to see it) 3) Do a search, noticing the ugly buttons from Comment 1 4) Apply patch and go back to a new search 5) Do a search again, notice no longer need to scroll to get to Search button and new, nicer buttons 6) Confirm they still work as expected Sponsored-by: Catalyst IT
Created attachment 61172 [details] [review] Bug 18078: Updating buttons for z39.50 Authority search 1) Go to Home > Authorities -> New from Z39.50 2) Notice how Search button is strangely off page (have to scroll down to see it) 3) Do a search, noticing the ugly buttons from Comment 1 4) Apply patch and go back to a new search 5) Do a search again, notice no longer need to scroll to get to Search button and new, nicer buttons 6) Confirm they still work as expected Sponsored-by: Catalyst IT Testplan changed to test with Authorities. Z39.50 search for authorities displays and Works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
I discussed with Owen about the use of btn-primary and I think the conclusion was not to use as it is, but customise our own. I do not find the bug report where it has been discussed, so ccing Owen to refresh my mind.
(In reply to Jonathan Druart from comment #3) > I discussed with Owen about the use of btn-primary and I think the > conclusion was not to use as it is, but customise our own. > I do not find the bug report where it has been discussed, so ccing Owen to > refresh my mind. Maybe Bug 18042 ?
Yes certainly, thanks!
No need to scroll to get to search buttons.
From bug 18042 comment 5: "- I think we should come up with updates to the default Bootstrap button styles if we are going to use more than the btn-default class. Bootstrap colors aren't designed to work with Koha's, and we should avoid looking like out-of-the-box Bootstrap."
Created attachment 82990 [details] [review] Bug 18078: Updating buttons for z39.50 Authority search This patch is an alternate proposal for updating buttons on the Z39.50 authority search popup window. It takes some aspects of the original proposal--Styling the pagination buttons with Bootstrap--and adds a new interaction for navigating to a specific page in results. To test, apply the patch and go to Authorities -> New Authority -> New from Z39.50/SRU. - Perform a search which will return multiple pages of results - At the bottom of the search results there should be a dropdown menu offering selection of page numbers. - Selecting a number from the dropdown should automatically load that page of results. - The current page number should be correctly pre-selected in the dropdown. - The "Previous" and "Next" buttons should work normally. - Perform a search which returns only a single page of results. There should be no navigation controls at the bottom of the page.
Wasn't able to complete the Search. After clicking Search, this error shows: Template process failed: undef error - WHILE loop terminated (> 1000 iterations) at /home/vagrant/kohaclone/C4/Templates.pm line 122 C4::Templates::output('C4::Templates=HASH(0x9b91f38)') called at /home/vagrant/kohaclone/cataloguing/z3950_auth_search.pl line 108 eval {...} at /home/vagrant/kohaclone/cataloguing/z3950_auth_search.pl line 2 CGI::Compile::ROOT::home_vagrant_kohaclone_cataloguing_z3950_auth_search_2epl::__ANON__('CGI::Compile=HASH(0x1493b08)', undef, '/home/vagrant/kohaclone/cataloguing/z3950_auth_search.pl', '/home/vagrant/kohaclone/cataloguing', 'ARRAY(0x958c750)') called at /usr/share/perl5/CGI/Compile.pm line 139 CGI::Compile::__ANON__ at /usr/share/perl5/CGI/Emulate/PSGI.pm line 29 CGI::Emulate::PSGI::__ANON__('HASH(0x9578700)') called at /usr/share/perl5/Plack/App/WrapCGI.pm line 87 Plack::App::WrapCGI::call('Plack::App::WrapCGI=HASH(0x95806a8)', 'HASH(0x9578700)') called at /usr/share/perl5/Plack/Component.pm line 50 Plack::Component::__ANON__('HASH(0x9578700)') called at /usr/share/perl5/Plack/App/CGIBin.pm line 50 Plack::App::CGIBin::serve_path('Plack::App::CGIBin=HASH(0x144ffd8)', 'HASH(0x9578700)', '/home/vagrant/kohaclone/cataloguing/z3950_auth_search.pl') called at /usr/share/perl5/Plack/App/File.pm line 34 Plack::App::File::call('Plack::App::CGIBin=HASH(0x144ffd8)', 'HASH(0x9578700)') called at /usr/share/perl5/Plack/Component.pm line 50 Plack::Component::__ANON__('HASH(0x9578700)') called at /usr/share/perl5/Plack/App/URLMap.pm line 71 Plack::App::URLMap::call('Plack::App::URLMap=HASH(0x8647598)', 'HASH(0x9578700)') called at /usr/share/perl5/Plack/Component.pm line 50 Plack::Component::__ANON__('HASH(0x9578700)') called at /home/vagrant/kohaclone/Koha/Middleware/SetEnv.pm line 104 Koha::Middleware::SetEnv::call('Koha::Middleware::SetEnv=HASH(0x94f9b98)', 'HASH(0x953e448)') called at /usr/share/perl5/Plack/Component.pm line 50 Plack::Component::__ANON__('HASH(0x953e448)') called at /usr/share/perl5/Plack/Middleware/Static.pm line 18 Plack::Middleware::Static::call('Plack::Middleware::Static=HASH(0x95451e8)', 'HASH(0x953e448)') called at /usr/share/perl5/Plack/Component.pm line 50 Plack::Component::__ANON__('HASH(0x953e448)') called at /usr/share/perl5/Plack/Middleware/ReverseProxy.pm line 68 Plack::Middleware::ReverseProxy::call('Plack::Middleware::ReverseProxy=HASH(0x9578808)', 'HASH(0x953e448)') called at /usr/share/perl5/Plack/Component.pm line 50 Plack::Component::__ANON__('HASH(0x953e448)') called at /usr/share/perl5/Plack/Middleware/Lint.pm line 24 Plack::Middleware::Lint::call('Plack::Middleware::Lint=HASH(0x1251178)', 'HASH(0x953e448)') called at /usr/share/perl5/Plack/Component.pm line 50 Plack::Middleware::StackTrace::try {...} at /usr/share/perl5/Try/Tiny.pm line 79 eval {...} at /usr/share/perl5/Try/Tiny.pm line 72 Plack::Middleware::StackTrace::call('Plack::Middleware::StackTrace=HASH(0x863b158)', 'HASH(0x953e448)') called at /usr/share/perl5/Plack/Component.pm line 50 Plack::Component::__ANON__('HASH(0x953e448)') called at /usr/share/perl5/Plack/Middleware/AccessLog.pm line 24 Plack::Middleware::AccessLog::call('Plack::Middleware::AccessLog=HASH(0x144ff48)', 'HASH(0x953e448)') called at /usr/share/perl5/Plack/Component.pm line 50 Plack::Component::__ANON__('HASH(0x953e448)') called at /usr/share/perl5/Plack/Util.pm line 142 eval {...} at /usr/share/perl5/Plack/Util.pm line 142 Plack::Util::run_app('CODE(0x94e42d0)', 'HASH(0x953e448)') called at /usr/share/perl5/Starman/Server.pm line 281 Starman::Server::process_request('Starman::Server=HASH(0xe8d6a8)', 'Net::Server::Proto::UNIX=GLOB(0x8647130)') called at /usr/share/perl5/Net/Server.pm line 74 Net::Server::run_client_connection('Starman::Server=HASH(0xe8d6a8)') called at /usr/share/perl5/Net/Server/PreFork.pm line 229 eval {...} at /usr/share/perl5/Net/Server/PreFork.pm line 229 Net::Server::PreFork::run_child('Starman::Server=HASH(0xe8d6a8)') called at /usr/share/perl5/Net/Server/PreFork.pm line 187 Net::Server::PreFork::run_n_children('Starman::Server=HASH(0xe8d6a8)', 2) called at /usr/share/perl5/Net/Server/PreFork.pm line 111 Net::Server::PreFork::loop('Starman::Server=HASH(0xe8d6a8)') called at /usr/share/perl5/Net/Server.pm line 60 Net::Server::run('Starman::Server=HASH(0xe8d6a8)', 'port', 'ARRAY(0x1394040)', 'host', '*', 'proto', 'tcp', 'serialize', 'none', 'min_servers', 2, 'min_spare_servers', 1, 'max_spare_servers', 1, 'max_servers', 2, 'max_requests', 50, 'user', 'kohadev-koha', 'group', 'kohadev-koha', 'listen', 1024, 'check_for_waiting', 1, 'no_client_stdout', 1, 'background', 1, 'pid_file', '/var/run/koha/kohadev/plack.pid', 'setsid', 1, 'log_file', '/var/log/koha/kohadev/plack-error.log') called at /usr/share/perl5/Starman/Server.pm line 89 Starman::Server::run('Starman::Server=HASH(0xe8d6a8)', 'CODE(0xe8d540)', 'HASH(0xe8d228)') called at /usr/share/perl5/Plack/Handler/Starman.pm line 18 Plack::Handler::Starman::run('Plack::Handler::Starman=HASH(0xe8d558)', 'CODE(0xe8d540)') called at /usr/share/perl5/Plack/Loader/Delayed.pm line 20 Plack::Loader::Delayed::run('Plack::Loader::Delayed=HASH(0xb94bf0)', 'Plack::Handler::Starman=HASH(0xe8d558)') called at /usr/share/perl5/Plack/Runner.pm line 277 Plack::Runner::run('Plack::Runner=HASH(0xb49b38)') called at /usr/bin/starman line 38
(In reply to Aleisha Amohia from comment #9) > Wasn't able to complete the Search. After clicking Search, this error shows: > > Template process failed: undef error - WHILE loop terminated (> 1000 > iterations) > at /home/vagrant/kohaclone/C4/Templates.pm line 122 I'm obsoleting that comment because it's really long and I think the above snippet is what's really relevant: My changed template tries to use a WHILE loop with result pages and there is a hard limit of 1000 for that kind of loop.
My patch is a non-starter, so I restored Aleisha's.
These buttons have also seen some updates with the staff interface redesign. It's a bit hard without a screenshot or similar, but I think main goals have been achieved.