[Bug 11411] New: Bootstrap theme (enhancement) for (right to left).
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Bug ID: 11411 Summary: Bootstrap theme (enhancement) for (right to left). Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: I18N/L10N Assignee: koha-bugs@lists.koha-community.org Reporter: karamqubsi@gmail.com QA Contact: testopia@bugs.koha-community.org CC: frederic@tamil.fr Created attachment 23615 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23615&action=edit Before the patch Hi, As you can see in the first attached image the welcome statement and the logout link is located to the right of the screen, but in the right to left languages it must be to the left. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 --- Comment #1 from Karam Qubsi <karamqubsi@gmail.com> --- Created attachment 23616 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23616&action=edit Bug-11411-ENH-Bootstrap-theme-enhancement-for right to left This patch will make a small change on the bootstrap-rtl.min.css file to make the welcome statement and the logout link located to the left of the screen. Test plan : 1-apply the patch 2-translate install ar-Arab . and that's it . -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Karam Qubsi <karamqubsi@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 --- Comment #2 from Karam Qubsi <karamqubsi@gmail.com> --- Created attachment 23617 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23617&action=edit After the patch The results must be as the attached image . -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #23616|0 |1 is obsolete| | --- Comment #3 from Chris Cormack <chris@bigballofwax.co.nz> --- Created attachment 23883 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23883&action=edit Bug 11411 [ENH] Bootstrap theme (enhancement) for (right to left) This patch will make a small change on the bootstrap-rtl.min.css file to make the welcome statement and the logout link located to the left of the screen. Test plan : 1-apply the patch 2-translate install ar-Arab 3-check the login/logut is on the left and that's it. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de Assignee|koha-bugs@lists.koha-commun |karamqubsi@gmail.com |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |In Discussion CC| |oleonard@myacpl.org --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- This works nicely, but not sure if it touches the necessary files. I see there is also /koha-tmpl/opac-tmpl/bootstrap/less/right-to-left.css. Adding Owen in cc as he will know and I hope for some hints :) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 --- Comment #5 from Owen Leonard <oleonard@myacpl.org> --- (In reply to Katrin Fischer from comment #4)
This works nicely, but not sure if it touches the necessary files. I see there is also /koha-tmpl/opac-tmpl/bootstrap/less/right-to-left.css.
Yeah there is a better way to handle this, although no one can be blamed for not knowing it because I haven't done a good job of laying out the guidelines. Two main points to consider: 1. No changes should be made to the files in bootstrap/lib unless they are "official upgrades" to these files, which are all created and maintained outside of the Koha project. If bootstrap-rtl.css has an error it and there isn't an "official upgrade" then the correction needs to be applied in Koha's custom RTL CSS. 2. All CSS in the Bootstrap theme is generated from LESS files (in bootstrap/less), so changes to CSS should be done in the LESS file. There is a specific rtl CSS file there: bootstrap/less/right-to-left.less, where this patch's changes should be made. Then the less file should be compiled and the resulting file put in bootstrap/css. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Owen, could you maybe provide a patch here? I am not sure anyone else understands how this works right now (including me) :) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #23883|0 |1 is obsolete| | --- Comment #7 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 24903 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=24903&action=edit Bug 11411 - Bootstrap theme (enhancement) for (right to left) This patch improves Koha's layout when a right-to-left language is being displayed by correcting a float set by the right-to-left version of Bootstrap CSS. The patch modifies the right-to-left Less file and includes a now-minified version of the compiled CSS. To test, install a right-to-left language like ar-Arab. Apply the patch and view the OPAC under the Bootstrap theme. When logged in, the aligned to the left. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |11601 Severity|enhancement |normal -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Holger Meißner <h.meissner.82@web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #24903|0 |1 is obsolete| | --- Comment #8 from Holger Meißner <h.meissner.82@web.de> --- Created attachment 25090 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25090&action=edit Bug 11411 - Bootstrap theme (enhancement) for (right to left) This patch improves Koha's layout when a right-to-left language is being displayed by correcting a float set by the right-to-left version of Bootstrap CSS. The patch modifies the right-to-left Less file and includes a now-minified version of the compiled CSS. To test, install a right-to-left language like ar-Arab. Apply the patch and view the OPAC under the Bootstrap theme. When logged in, the aligned to the left. Signed-off-by: Holger Meißner <h.meissner.82@web.de> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Holger Meißner <h.meissner.82@web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |h.meissner.82@web.de --- Comment #9 from Holger Meißner <h.meissner.82@web.de> --- Works as described. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25090|0 |1 is obsolete| | CC| |kyle@bywatersolutions.com --- Comment #10 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 25131 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25131&action=edit Bug 11411 - Bootstrap theme (enhancement) for (right to left) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 --- Comment #11 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 25132 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25132&action=edit Bug 11411 [QA Followup] - Tidy css to restore readability -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25131|0 |1 is obsolete| | CC| |gmcharlt@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25132|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25090|1 |0 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11411 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #12 from Galen Charlton <gmcharlt@gmail.com> --- I have pushed Owen's original patch, but I'm declining to push the follow-ups supplied by Kyle because they defeat the (network performance) purpose of the minification. Also, in this case I don't think we want the compiled CSS to be readable, since one /should/ be editing the LESS files. That said, I'm not setting this decision in concrete, as only time will tell whether the benefits of minifying the CSS file outweigh the stumbling blocks for folks using a browser CSS debugger. However, I will point out that Chrome, at least, includes a pretty-print function in its debugger. Thanks, Owen! -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org