Summary: | offset is wrong on plugins-disabled.tt page | ||
---|---|---|---|
Product: | Koha | Reporter: | Lucas Gass (lukeg) <lucas> |
Component: | Staff interface | Assignee: | Lucas Gass (lukeg) <lucas> |
Status: | Pushed to stable --- | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | trivial | ||
Priority: | P5 - low | CC: | david, gmcharlt, martin.renvoize, wainuiwitikapark |
Version: | Main | Keywords: | Academy |
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: |
This fixes the display of the plugins page, when plugins are disabled in koha-conf.xml and the page is accessed directly. The message that plugins are disabled is now indented, instead of aligned to the far left.
|
|
Version(s) released in: |
25.05.00, 24.11.02
|
Circulation function: | |
Attachments: |
Bug 38367: Fix offset on plugin-dsiabled.tt
Bug 38367: Fix offset on plugin-dsiabled.tt Bug 38367: Fix offset on plugin-dsiabled.tt Bug 38367: Fix offset on plugin-dsiabled.tt |
Description
Lucas Gass (lukeg)
2024-11-06 00:10:30 UTC
Created attachment 174055 [details] [review] Bug 38367: Fix offset on plugin-dsiabled.tt To test: 1. Go to your koha-conf.xml and turn plugins off: <enable_plugins>0</enable_plugins> 2. Go to Administration -> Plugins 3. The text is not correctly offset. ( Bootstrap class is col-sm-offset-3 and should be offset-sm-3 in Bootstrap 5 ) 4. APPLY PATCH 5. Try again, the offset should look right. The updated markup has "col-sm-6 offset-sm-3 col-md-6 col-md-offset-3", which still includes the old "col-md-offset-3" class. The previous markup was redundant, because it's the same width and the same offset for both the sm and md breakpoint. We could make it simply "col-sm-6 offset-sm-3" or include a different breakpoint for md, e.g. "col-sm-6 offset-sm-3 col-md-10 offset-md-1" Created attachment 174085 [details] [review] Bug 38367: Fix offset on plugin-dsiabled.tt To test: 1. Go to your koha-conf.xml and turn plugins off: <enable_plugins>0</enable_plugins> 2. Go to Administration -> Plugins 3. The text is not correctly offset. ( Bootstrap class is col-sm-offset-3 and should be offset-sm-3 in Bootstrap 5 ) 4. APPLY PATCH 5. Try again, the offset should look right. I had a go at testing, but I can't get this to work. If you set <enable_plugins>0</enable_plugins>, then there is no Administration > Plugins. Could you please update the test plan to reflect how to replicate/demonstrate the issue? You have to navigate directly to http://127.0.0.1:8081/cgi-bin/koha/plugins/plugins-home.pl Created attachment 175033 [details] [review] Bug 38367: Fix offset on plugin-dsiabled.tt To test: 1. Go to your koha-conf.xml and turn plugins off: <enable_plugins>0</enable_plugins> 2. Go to Administration -> Plugins 3. The text is not correctly offset. ( Bootstrap class is col-sm-offset-3 and should be offset-sm-3 in Bootstrap 5 ) 4. APPLY PATCH 5. Try again, the offset should look right. Signed-off-by: David Nind <david@davidnind.com> (In reply to Owen Leonard from comment #5) > You have to navigate directly to > http://127.0.0.1:8081/cgi-bin/koha/plugins/plugins-home.pl Thanks Owen! Created attachment 175439 [details] [review] Bug 38367: Fix offset on plugin-dsiabled.tt To test: 1. Go to your koha-conf.xml and turn plugins off: <enable_plugins>0</enable_plugins> 2. Go to Administration -> Plugins 3. The text is not correctly offset. ( Bootstrap class is col-sm-offset-3 and should be offset-sm-3 in Bootstrap 5 ) 4. APPLY PATCH 5. Try again, the offset should look right. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Pushed for 25.05! Well done everyone, thank you! This is in the commit log for 24.11, though not noted here yet. Not backporting to 24.05.x unless requested This was pushed for 24.11.02 |