Summary: | Tabs on Additional Content page need space above | ||
---|---|---|---|
Product: | Koha | Reporter: | Martin Renvoize (ashimema) <martin.renvoize> |
Component: | Staff interface | Assignee: | Owen Leonard <oleonard> |
Status: | RESOLVED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | fridolin.somers, gmcharlt |
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: |
23.11.00,23.05.05,22.11.11
|
|
Circulation function: | |||
Attachments: |
Screenshot
Bug 34921: Tabs on Additional Content page need space above Bug 34921: Tabs on Additional Content page need space above |
Description
Martin Renvoize (ashimema)
2023-09-26 15:41:21 UTC
Created attachment 156236 [details]
Screenshot
Created attachment 156286 [details] [review] Bug 34921: Tabs on Additional Content page need space above This patch adds a separator element, <hr /> between the additional contents form and the tabs for visual clarity. Using an element is a little hackish but it's a simple solution. To test, apply the patch and go to Tools -> News -> New entry. There should be a clear gap between the top form and the tabs below. Created attachment 156288 [details] [review] Bug 34921: Tabs on Additional Content page need space above This patch adds a separator element, <hr /> between the additional contents form and the tabs for visual clarity. Using an element is a little hackish but it's a simple solution. To test, apply the patch and go to Tools -> News -> New entry. There should be a clear gap between the top form and the tabs below. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Your right.. it feels a little hacky, but is also the least intrusive and least likely to have unexpected side effects. Passing QA Pushed to master for 23.11. Nice work everyone, thanks! Pushed to 23.05.x for 23.05.05 Nice work everyone! Pushed to oldstable for 22.11.x |