Bug 22287

Summary: Correct new charts CSS
Product: Koha Reporter: Owen Leonard <oleonard>
Component: ReportsAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: cslone, fridolin.somers, katrin.fischer, lucas, martin.renvoize, nick
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 22168    
Bug Blocks:    
Attachments: Bug 22287: Correct new charts CSS
Bug 22287: Correct new charts CSS
Bug 22287: Correct new charts CSS

Description Owen Leonard 2019-02-07 13:10:25 UTC
Bug 22168 introduced a change to reports CSS which had an unintended effect on the SQL reports edit form.
Comment 1 Owen Leonard 2019-02-07 13:17:49 UTC Comment hidden (obsolete)
Comment 2 Mikaël Olangcay Brisebois 2019-02-08 19:34:32 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2019-02-15 06:40:41 UTC
Created attachment 85140 [details] [review]
Bug 22287: Correct new charts CSS

This patch reverts some changes to reports CSS from Bug 22168 which were
unnecessary and which adversely affected the report add/edit form
styling.

To test, apply the patch and clear your browser cache if necessary.

Go to Reports -> Saved reports and repeat the test plan for Bug 22168:

- Run a report which returns more than one column.
- On the report results page, click the "Create chart" button. The chart
  settings form should appear in a modal window.
- Click the "Draw" button. The modal should disappear and the chart
  should be shown.

Then test the process for adding or editing an SQL report. The form
should be styled in a way consistent with other forms in the staff
client.

Signed-off-by: Mikaël Olangcay Brisebois <mikael.olangcay-brisebois@inLibro.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Nick Clemens (kidclamp) 2019-02-15 18:51:11 UTC
Awesome work all!

Pushed to master for 19.05
Comment 5 Martin Renvoize 2019-02-25 13:38:41 UTC
Pushed to 18.11.x for 18.11.04
Comment 6 Lucas Gass 2019-03-05 15:23:26 UTC
missing 22168 in 18.05.x series. won't backport