Merge branch 'wip_MDL-60250_M33' of git://github.com/gjb2048/moodle i…

…nto MOODLE_33_STABLE
moodle · Oct 25, 2017 · 8e14926847011c37d50590f65ed9142617268ae2 · 8e14926
2 parents 32e3f25 + 4e2d813
commit 8e14926847011c37d50590f65ed9142617268ae2
Unified Split

Showing with 9 additions and 0 deletions.

+1 −0 theme/bootstrapbase/less/bootstrap/popovers.less

+7 −0 theme/bootstrapbase/readme_moodle.txt

+1 −0 theme/bootstrapbase/style/moodle.css
diff --git a/theme/bootstrapbase/less/bootstrap/popovers.less b/theme/bootstrapbase/less/bootstrap/popovers.less
@@ -6,6 +6,7 @@
 .popover {
   position: absolute;
   top: 0;
+  /*rtl:ignore*/
   left: 0;
   z-index: @zindexPopover;
   display: none;

diff --git a/theme/bootstrapbase/readme_moodle.txt b/theme/bootstrapbase/readme_moodle.txt
@@ -51,3 +51,10 @@ in the compiled CSS (MDL-53152):
 * @gridRowWidth
 * @gridRowWidth1200
 * @gridRowWidth768
+
+popovers.less
+-------------
+MDL-60250 - The '/*rtl:ignore*/' directive has been added to the 'left' attribute of the 'popover' class so that
+the rtlcss-php tool (https://github.com/moodlehq/rtlcss-php) does not flip this to a 'right' attribute and cause
+the popover to be misplaced on the page when the JavaScript calculates the postion of the popover and adds an
+overriding inline CSS 'left' attribute which is fine in LTR languages but confuses it in RTL languages.
diff --git a/theme/bootstrapbase/style/moodle.css b/theme/bootstrapbase/style/moodle.css
@@ -14906,6 +14906,7 @@ input[type="submit"].btn.btn-mini {
 .popover {
   position: absolute;
   top: 0;
+  /*rtl:ignore*/
   left: 0;
   z-index: 1010;
   display: none;