Created attachment 76039 [details] Screenshot showing system preferences standard view I think the system preferences page could use some work to make it more readable. My proposal does this: - Restructure the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned.
Created attachment 76040 [details] Screenshot showing system preferences filtered using in-page filter
Created attachment 76041 [details] Screenshot showing system preferences search results
Created attachment 76097 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct.
Created attachment 76106 [details] Screenshot of the system preferences page
When I apply the patch, it doesn't look the same as the screenshots. The section headings are blue and the syspref names are almost the same as their description. making it hard to read. I like it, but would like the syspref names to be at least in bold. Also, would it be possible to keep the same indent as the first line on the 2nd + line of the description? For example, in my screenshot, in the description for gist, the second line, starting at "rounded" doesn't have the same indent as the line above. I think it makes it look messy. I would miss the arrows to collapse a section (when they're really long, it saves scrolling energy). Would it still be possible to have them in this new design? It's not the end of the world if it's not
(In reply to Caroline Cyr La Rose from comment #5) > When I apply the patch, it doesn't look the same as the screenshots. Did you clear your browser cache? That could explain the difference in what you see.
Comment on attachment 76097 [details] [review] Bug 20935: Redesign system preferences interface Review of attachment 76097 [details] [review]: ----------------------------------------------------------------- ::: koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences.tt @@ +156,5 @@ > + </div> <!-- /.col-sm-10.col-sm-push-2 --> > + > + <div class="col-sm-2 col-sm-pull-10"> > + <aside> > + [% INCLUDE 'admin-menu.inc' %] should be prefs-menu
Also, the "Local use" preferences are not addressed in this patch But great work Owen, I like this style!
* I would not add the alert box when nothing is found * Left navigation is replaced by the admin one, instead of the pref specific one (prefs-menu.inc vs admin-menu.inc), I do not think it's expected.
Created attachment 76548 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct.
(In reply to Josef Moravec from comment #8) > Also, the "Local use" preferences are not addressed in this patch The local use tab wasn't updated in the last round of interface changes, so I didn't think it was a priority.
Created attachment 76664 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct. Followed test plan and pleased to say everything worked as expected. Looking forward to this enhancement. Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au>
Just some small remarks, no blockers: Instead of "Click to edit" and "Click to collapse" I think it should just be "Edit" and "Collapse" as there is no need to extra label a link for clicking. Also... I think the German translation would be horribly long :) Attaching a patch with the suggested change, feel free to disagree!
Created attachment 76754 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct. Followed test plan and pleased to say everything worked as expected. Looking forward to this enhancement. Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 76755 [details] [review] Bug 20935: Remove "Click to" from some links Changes "Click to edit" and "Click to collapse" to just "Edit" and "Collapse" as the "Click to" appears not necessary for elements formatted as links.
There is no question that this is more modern and prettier looking than before. The new filter functionality is a great addition, as are the "(modified)" labels on prefs changed. I am still a little torn on usability. I have the impression that it takes me a little longer to find things with the new styling. In the old interface the prefs were (ugly) Courier and in one column, so scanning over the page was pretty fast. This might be just a thing of 'being used to the old way'. I'd love another opinion as this is a bigger change and it would be good to get another set of eyes on it. Could be another sign-off or QA opinion.
When there is a match: https://screenshots.firefox.com/iaqfOGIV1yWAylQg/pro.kohadev.org When there is no match: https://screenshots.firefox.com/iaqfOGIV1yWAylQg/pro.kohadev.org The "Acquisitions preferences" h2 is display under the toolbar (I have selected the text to show it on the screenshot.
Created attachment 77163 [details] [review] Bug 20935: Remove 'no match' box when it does Test plan: Hit /cgi-bin/koha/admin/preferences.pl Search for "AcqCreateX" => You get "No matching terms were found on this page" Remove the last X => Without this patch the "no matching" box is still displayed
Found and fix an ergonomic bug with the toolbar on bug 21099.
(In reply to Jonathan Druart from comment #17) > When there is a match: > https://screenshots.firefox.com/iaqfOGIV1yWAylQg/pro.kohadev.org > > When there is no match: > https://screenshots.firefox.com/iaqfOGIV1yWAylQg/pro.kohadev.org > > The "Acquisitions preferences" h2 is display under the toolbar (I have > selected the text to show it on the screenshot. I have no idea how to fix that. I have also noticed that fixFloat duplicate the toolbar node (one is hidden), and so we should not have an id on them. Passing QA as it is a minor ergonomic issue.
Created attachment 77165 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct. Followed test plan and pleased to say everything worked as expected. Looking forward to this enhancement. Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 77166 [details] [review] Bug 20935: Remove "Click to" from some links Changes "Click to edit" and "Click to collapse" to just "Edit" and "Collapse" as the "Click to" appears not necessary for elements formatted as links.
Created attachment 77167 [details] [review] Bug 20935: Remove 'no match' box when it does Test plan: Hit /cgi-bin/koha/admin/preferences.pl Search for "AcqCreateX" => You get "No matching terms were found on this page" Remove the last X => Without this patch the "no matching" box is still displayed
Created attachment 77707 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct. Followed test plan and pleased to say everything worked as expected. Looking forward to this enhancement. Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 77708 [details] [review] Bug 20935: Remove "Click to" from some links Changes "Click to edit" and "Click to collapse" to just "Edit" and "Collapse" as the "Click to" appears not necessary for elements formatted as links. Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 77709 [details] [review] Bug 20935: Remove 'no match' box when it does Test plan: Hit /cgi-bin/koha/admin/preferences.pl Search for "AcqCreateX" => You get "No matching terms were found on this page" Remove the last X => Without this patch the "no matching" box is still displayed Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 77844 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct. Followed test plan and pleased to say everything worked as expected. Looking forward to this enhancement. Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 77845 [details] [review] Bug 20935: Remove "Click to" from some links Changes "Click to edit" and "Click to collapse" to just "Edit" and "Collapse" as the "Click to" appears not necessary for elements formatted as links. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 77846 [details] [review] Bug 20935: Remove 'no match' box when it does Test plan: Hit /cgi-bin/koha/admin/preferences.pl Search for "AcqCreateX" => You get "No matching terms were found on this page" Remove the last X => Without this patch the "no matching" box is still displayed Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
I have found a small regression added by the toolbar changes (bug 21139), RM please push the follow-up I have attached on it along with these patches.
(In reply to Jonathan Druart from comment #20) > (In reply to Jonathan Druart from comment #17) > > When there is a match: > > https://screenshots.firefox.com/iaqfOGIV1yWAylQg/pro.kohadev.org > > > > When there is no match: > > https://screenshots.firefox.com/iaqfOGIV1yWAylQg/pro.kohadev.org > > > > The "Acquisitions preferences" h2 is display under the toolbar (I have > > selected the text to show it on the screenshot. > > I have no idea how to fix that. I have also noticed that fixFloat duplicate > the toolbar node (one is hidden), and so we should not have an id on them. > > Passing QA as it is a minor ergonomic issue. Hum, I have posted twice the same link! When there is no match: https://screenshots.firefox.com/E2ey9Akb68rPHngV/pro.kohadev.org This is still valid, but minor and can be pushed as it IMO.
Please rebase on 13618
Created attachment 79177 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct. Followed test plan and pleased to say everything worked as expected. Looking forward to this enhancement. Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 79178 [details] [review] Bug 20935: Remove "Click to" from some links Changes "Click to edit" and "Click to collapse" to just "Edit" and "Collapse" as the "Click to" appears not necessary for elements formatted as links. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 79179 [details] [review] Bug 20935: Remove 'no match' box when it does Test plan: Hit /cgi-bin/koha/admin/preferences.pl Search for "AcqCreateX" => You get "No matching terms were found on this page" Remove the last X => Without this patch the "no matching" box is still displayed Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Hi Owen, what happened here?
(In reply to Katrin Fischer from comment #36) > Hi Owen, what happened here? I wanted to try to address some comments and suggestions I got during Hackfest.
Created attachment 82884 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct. Followed test plan and pleased to say everything worked as expected. Looking forward to this enhancement. Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 82885 [details] [review] Bug 20935: Remove "Click to" from some links Changes "Click to edit" and "Click to collapse" to just "Edit" and "Collapse" as the "Click to" appears not necessary for elements formatted as links. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 82886 [details] [review] Bug 20935: Remove 'no match' box when it does Test plan: Hit /cgi-bin/koha/admin/preferences.pl Search for "AcqCreateX" => You get "No matching terms were found on this page" Remove the last X => Without this patch the "no matching" box is still displayed Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 82887 [details] [review] Bug 20935: Add in-page links to the sidebar This patch adds links to the sidebar menu which allow the user to jump to a section within the page.
I was asked to add alternate-row highlighting to this but I found that it made it less readable rather than more. So I didn't do that.
Created attachment 82888 [details] [review] Bug 20935: Add missing html filter
Some comments I made on irc, but thought should be recorded here. Functionality wise I can't find any issues, it all works as expected. However: * I think we need to make the left menu float as you can no longer collapse preference sections and instead use the left menu more frequently to navigate between sections. * Somehow although it's spaced out more, it feels less readable and clear.. we're highlighting the non-descriptive in many cases syspref codes/names but making the actual descriptions and options a bit smaller. * Sometimes it's a good thing to have more on the page, this change introduces a not-insignificant descrease in data per page and as such results in more scrolling to find the preference you want (assuming you can't remember it's name enough to search / filter for it. I really want to like this update and I do think it looks more modern, but I can't happily go for it right now I'm afraid. Hugs to Owen for giving it a good try.. I look forward to more feedback being given and perhaps being over-ruled on this one.. It would be a great area to smarten up but I don't think we're quit there yet.
Created attachment 86784 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct. Followed test plan and pleased to say everything worked as expected. Looking forward to this enhancement. Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 86785 [details] [review] Bug 20935: Remove "Click to" from some links Changes "Click to edit" and "Click to collapse" to just "Edit" and "Collapse" as the "Click to" appears not necessary for elements formatted as links. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 86786 [details] [review] Bug 20935: Remove 'no match' box when it does Test plan: Hit /cgi-bin/koha/admin/preferences.pl Search for "AcqCreateX" => You get "No matching terms were found on this page" Remove the last X => Without this patch the "no matching" box is still displayed Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 86787 [details] [review] Bug 20935: Add in-page links to the sidebar This patch adds links to the sidebar menu which allow the user to jump to a section within the page.
Created attachment 86788 [details] [review] Bug 20935: Add missing html filter
Rebase only.
Created attachment 86833 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct. Followed test plan and pleased to say everything worked as expected. Looking forward to this enhancement. Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 86834 [details] [review] Bug 20935: Remove "Click to" from some links Changes "Click to edit" and "Click to collapse" to just "Edit" and "Collapse" as the "Click to" appears not necessary for elements formatted as links. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 86835 [details] [review] Bug 20935: Remove 'no match' box when it does Test plan: Hit /cgi-bin/koha/admin/preferences.pl Search for "AcqCreateX" => You get "No matching terms were found on this page" Remove the last X => Without this patch the "no matching" box is still displayed Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 86836 [details] [review] Bug 20935: Add in-page links to the sidebar This patch adds links to the sidebar menu which allow the user to jump to a section within the page. Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 86837 [details] [review] Bug 20935: Add missing html filter Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Michal Denar <black23@gmail.com>
Comment on attachment 86837 [details] [review] Bug 20935: Add missing html filter Review of attachment 86837 [details] [review]: ----------------------------------------------------------------- ::: koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences.tt @@ +38,4 @@ > <div id="toolbar"> > Results in sections: > [% FOREACH TAB IN TABS %] > + <a href="#[% TAB.tab_title | url %]">[% TAB.tab_title | html %]</a>[% UNLESS ( loop.last ) %] • [% END %] In fragment you should use uri filter, not url.
I have question about multiselect fields: - Should the text be blue? I think we could stay with black here - The width of multiselect field is set bad, so there is scrollbar at the bottom, is it possible to do something with that? See screenshot: https://prnt.sc/n0u6ji Marking Failed QA means I need answer here. Otherwise it is looking good. (Just the small issue with uri/url filter)
Created attachment 89936 [details] [review] Bug 20935: Redesign system preferences interface This patch proposes a new interface for system preferences. The goal is to make the interface more readable and to give the user additional options for finding what they need. - Restructures the preferences list as a <dl>, with the preference name as the <dt> and the description as the <dd>. - Add a JS filter library (mark.js) to do live filtering of the page and add search term highlighting. - When the user has searched, show links to the sections in which results were returned. To test, apply the patch and go to Administration -> System preferences. - Test the functionality of setting various kinds of preferences, including multi-selects, textareas, WYSIWYG editors, etc. - Test the filter form at the top of the page for narrowing in-page results. - Test both successful and unsuccessful filters. - Perform a system preference search. - Test the in-page links to the different sections. - Test that in-page filtering works in this case too. - View the licenses tab of the "About" page and check that the information about mark.js looks correct. Followed test plan and pleased to say everything worked as expected. Looking forward to this enhancement. Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 89937 [details] [review] Bug 20935: Remove "Click to" from some links Changes "Click to edit" and "Click to collapse" to just "Edit" and "Collapse" as the "Click to" appears not necessary for elements formatted as links. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 89938 [details] [review] Bug 20935: Remove 'no match' box when it does Test plan: Hit /cgi-bin/koha/admin/preferences.pl Search for "AcqCreateX" => You get "No matching terms were found on this page" Remove the last X => Without this patch the "no matching" box is still displayed Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 89939 [details] [review] Bug 20935: Add in-page links to the sidebar This patch adds links to the sidebar menu which allow the user to jump to a section within the page. Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 89940 [details] [review] Bug 20935: Add missing html filter Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Michal Denar <black23@gmail.com>