Summary: | Reports menu 'Show SQL code' wrong border radius | ||
---|---|---|---|
Product: | Koha | Reporter: | Fridolin Somers <fridolin.somers> |
Component: | Reports | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Tomás Cohen Arazi <tomascohen> |
Severity: | minor | ||
Priority: | P5 - low | CC: | andrewfh, kyle, nugged, oleonard, tomascohen, victor |
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: |
22.05.00,21.11.02,21.05.08
|
|
Attachments: |
Bug 29680: Change markup to correct button radius
Bug 29680: Change markup to correct button radius Bug 29680: Change markup to correct button radius Bug 29680: Change markup to correct button radius |
Description
Fridolin Somers
2021-12-10 20:38:48 UTC
Created attachment 128466 [details] [review] Bug 29680: Change markup to correct button radius This patch alters the markup around the "Show SQL/Hide SQL" buttons so that they are not styled as if they are a button group. To test, apply the patch and run an SQL report. The "Show SQL code" button should be styled consistently with other single buttons in the toolbar. Click it to test that the "Hide SQL code" button is also correct. Created attachment 128579 [details] [review] Bug 29680: Change markup to correct button radius This patch alters the markup around the "Show SQL/Hide SQL" buttons so that they are not styled as if they are a button group. To test, apply the patch and run an SQL report. The "Show SQL code" button should be styled consistently with other single buttons in the toolbar. Click it to test that the "Hide SQL code" button is also correct. Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Created attachment 128659 [details] [review] Bug 29680: Change markup to correct button radius This patch alters the markup around the "Show SQL/Hide SQL" buttons so that they are not styled as if they are a button group. To test, apply the patch and run an SQL report. The "Show SQL code" button should be styled consistently with other single buttons in the toolbar. Click it to test that the "Hide SQL code" button is also correct. Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Andreas Roussos <a.roussos@dataly.gr> Created attachment 128677 [details] [review] Bug 29680: Change markup to correct button radius This patch alters the markup around the "Show SQL/Hide SQL" buttons so that they are not styled as if they are a button group. To test, apply the patch and run an SQL report. The "Show SQL code" button should be styled consistently with other single buttons in the toolbar. Click it to test that the "Hide SQL code" button is also correct. Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Andreas Roussos <a.roussos@dataly.gr> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Pushed to master for 22.05, thanks to everybody involved 🦄 Pushed to 21.11.x for 21.11.02 Pushed to 21.05.x for 21.05.08 Not backported to oldoldstable (20.11.x). Feel free to ask if it's needed. |