Bug 18283

Summary: Display improvements on report results - hide code and change wording
Product: Koha Reporter: Nick Clemens <nick>
Component: ReportsAssignee: Nick Clemens <nick>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: cbrannon, jonathan.druart, katrin.fischer, kyle, veron
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 29312    
Attachments: Bug 18283 - Display improvements on report results - hide code and change wording
Bug 18283 - Display improvements on report results - hide code and change wording
Bug 18283 - Display improvements on report results - hide code and change wording
Bug 18283 - Display improvements on report results - hide code and change wording
Bug 18283 - Display improvements on report results - hide code and change wording
[SIGNED OFF] Bug 18283 - Display improvements on report results - hide code and change wording
Bug 18283 - Display improvements on report results - hide code and change wording
Bug 18283: 'sql' should be 'SQL'

Description Nick Clemens 2017-03-16 15:34:04 UTC

    
Comment 1 Nick Clemens 2017-03-16 15:39:13 UTC Comment hidden (obsolete)
Comment 2 Christopher Brannon 2017-03-16 19:01:35 UTC Comment hidden (obsolete)
Comment 3 Nick Clemens 2017-03-16 20:33:57 UTC Comment hidden (obsolete)
Comment 4 Marc Véron 2017-03-16 21:19:44 UTC Comment hidden (obsolete)
Comment 5 Nick Clemens 2017-03-17 10:05:06 UTC Comment hidden (obsolete)
Comment 6 Christopher Brannon 2017-03-17 17:05:27 UTC Comment hidden (obsolete)
Comment 7 Nick Clemens 2017-03-20 08:14:27 UTC
Created attachment 61249 [details] [review]
Bug 18283 - Display improvements on report results - hide code and change wording

This patch makes small display improvements to the report results page.

-Add toggle link for SQL code
-Change results count wording
-Add 'Notes:' label report notes

To test:
1 - Apply patch
2 - Run some reports
3 - Check that results page makes sense
4 - Hide and display code and ensure toggle works
Comment 8 Marc Véron 2017-03-20 09:45:23 UTC
Created attachment 61253 [details] [review]
Bug 18283 - Display improvements on report results - hide code and change wording

This patch makes small display improvements to the report results page.

-Add toggle link for SQL code
-Change results count wording
-Add 'Notes:' label report notes

To test:
1 - Apply patch
2 - Run some reports
3 - Check that results page makes sense
4 - Hide and display code and ensure toggle works

Followed test plan, works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>
Comment 9 Christopher Brannon 2017-03-20 14:50:10 UTC
Created attachment 61318 [details] [review]
[SIGNED OFF] Bug 18283 - Display improvements on report results - hide code and change wording

This patch makes small display improvements to the report results page.

-Add toggle link for SQL code
-Change results count wording
-Add 'Notes:' label report notes

To test:
1 - Apply patch
2 - Run some reports
3 - Check that results page makes sense
4 - Hide and display code and ensure toggle works

Followed test plan, works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org>
Comment 10 Jonathan Druart 2017-03-20 19:45:06 UTC
Why is this supposed to be an improvement? What are the use cases?
Comment 11 Christopher Brannon 2017-03-20 21:44:59 UTC
(In reply to Jonathan Druart from comment #10)
> Why is this supposed to be an improvement? What are the use cases?

I can tell you that at our library, we have jquery setup that does something similar.  It bugged me the moment we started using Koha that the SQL was showing.  Your average staff person doesn't care about it, and depending on the size of your sql, you might have to scroll down a bit to see results.  It's ugly.  It makes sense to hide it, but be able to reveal it when needed.
Comment 12 Jonathan Druart 2017-03-21 10:31:06 UTC
Created attachment 61359 [details] [review]
Bug 18283 - Display improvements on report results - hide code and change wording

This patch makes small display improvements to the report results page.

-Add toggle link for SQL code
-Change results count wording
-Add 'Notes:' label report notes

To test:
1 - Apply patch
2 - Run some reports
3 - Check that results page makes sense
4 - Hide and display code and ensure toggle works

Followed test plan, works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 13 Jonathan Druart 2017-03-21 10:31:10 UTC
Created attachment 61360 [details] [review]
Bug 18283: 'sql' should be 'SQL'

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 14 Kyle M Hall 2017-03-31 14:51:15 UTC
Pushed to master for 17.05, thanks Nick, Jonathan!
Comment 15 Katrin Fischer 2017-04-02 17:45:37 UTC
This won't get ported back to 16.11.x as it is an enhancement.