?
avatar mbabker
mbabker
13 Sep 2016

Steps to reproduce the issue

Review https://github.com/joomla/joomla-cms/blob/3.6.2/media/jui/less/bootstrap-rtl.less#L34-L50

Expected result

Grid widths should be calculated based on the variables defined by a template.

Actual result

Grid widths are hardcoded causing inconsistent display when RTL is in use.

System information (as much as possible)

N/A

Additional comments

To demonstrate the issues this causes, go to https://issues.joomla.org/markdown and view the page with the Arabic language enabled. Inspect the page and find one of the span6 elements. Note the widths coming from the RTL CSS file (which is compiled using the core Bootstrap file; the grid widths are NOT overridden by the joomla.org site template) and note the values it overrides from the "normal" CSS files. If variables were in use, the right margin for the RTL CSS would be the correct 2.564102564102564% value versus the hardcoded 2.127659574%.

avatar mbabker mbabker - open - 13 Sep 2016
avatar zero-24 zero-24 - change - 14 Sep 2016
Labels Added: ?
avatar mbabker
mbabker - comment - 14 Sep 2016

Nevermind. Closing as can't fix without B/C breaks. Basically every template is going to need their own RTL LESS and will need to copy the core file (if they want to use it as a starting point) to be able to pass variables in. It should be noted though that the default RTL LESS and CSS are faulty without being fixable.

avatar mbabker mbabker - change - 14 Sep 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-09-14 13:53:17
Closed_By mbabker
avatar mbabker mbabker - close - 14 Sep 2016
avatar brianteeman brianteeman - close - 14 Sep 2016
avatar brianteeman brianteeman - change - 14 Sep 2016
Labels Added: ?
avatar zero-24 zero-24 - change - 20 Jan 2017
Labels Removed: ?
avatar zero-24 zero-24 - unlabeled - 20 Jan 2017

Add a Comment

Login with GitHub to post a comment