Bug 22287: Correct new charts CSS
authorOwen Leonard <oleonard@myacpl.org>
Thu, 7 Feb 2019 13:11:35 +0000 (13:11 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Fri, 15 Feb 2019 18:42:47 +0000 (18:42 +0000)
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>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/css/reports.css

index 2152cc8..b938b7b 100644 (file)
@@ -74,23 +74,11 @@ del {
     color: #C00;
 }
 
-fieldset.rows {
-    background-color: transparent;
-    border: 0;
-    font-size: 100%;
-    margin: 0;
-    padding: 0;
-}
-
 fieldset.rows label,
 fieldset.rows span.label {
     width: 12em;
 }
 
-fieldset.rows legend {
-    margin-left: 0;
-}
-
 fieldset.rows .column_config_row {
     display: flex;
     flex-wrap: wrap;
@@ -108,4 +96,8 @@ fieldset.rows .chart-column-conf label {
     margin: .4em 0;
     text-align: left;
     width: auto;
+}
+
+#makechart select {
+    max-width: 12em;
 }
\ No newline at end of file