Bug 25974

Summary: Remove inline style from table settings administration page
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: alex.arnaud, jonathan.druart, lucas, mark.gavillet
Version: master   
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:
20.11.00
Bug Depends on: 24156    
Bug Blocks:    
Attachments: Bug 25974: Remove inline style from table settings administration page
Bug 25974: Remove inline style from table settings administration page
Bug 25974: Remove inline style from table settings administration page

Description Owen Leonard 2020-07-10 14:27:53 UTC
Bug 24156 added some inline styles to columns_settings.tt which seem out of place. I'd prefer to remove the inline style and add some subtler CSS to the page.
Comment 1 Owen Leonard 2020-07-10 14:40:42 UTC
Created attachment 106770 [details] [review]
Bug 25974: Remove inline style from table settings administration page

This patch removes an inline style attribute from the table settings
administration template. A different style is added to the page's
<style> block to try to accomplish a similar effect of expressing the
hierarchy of the page.

To test, apply the patch and go to Administration -> Table settings.

The individual tables of settings should be offset from the headings
above them.
Comment 2 David Nind 2020-07-11 20:47:37 UTC
Created attachment 106786 [details] [review]
Bug 25974: Remove inline style from table settings administration page

This patch removes an inline style attribute from the table settings
administration template. A different style is added to the page's
<style> block to try to accomplish a similar effect of expressing the
hierarchy of the page.

To test, apply the patch and go to Administration -> Table settings.

The individual tables of settings should be offset from the headings
above them.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 Katrin Fischer 2020-07-13 21:26:02 UTC
Created attachment 106853 [details] [review]
Bug 25974: Remove inline style from table settings administration page

This patch removes an inline style attribute from the table settings
administration template. A different style is added to the page's
<style> block to try to accomplish a similar effect of expressing the
hierarchy of the page.

To test, apply the patch and go to Administration -> Table settings.

The individual tables of settings should be offset from the headings
above them.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Jonathan Druart 2020-07-20 12:08:26 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 5 Lucas Gass 2020-07-24 23:09:51 UTC
missing a dependency for 20.05.x, no backport