Moodle

including a YUI CSS file breaks ajax editing in IE6

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9.3
  • Fix Version/s: 1.9.6
  • Component/s: AJAX
  • Labels:
    None
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

the file \lib\yui\container\assets\skins\sam\container.css

contains the following CSS selector, which makes all the select boxes on the page disappear. Might be a YUI bug, but I had to comment it to make things work. (no line number as the file is minified)

.masked select,.drag select,.hide-select select{_visibility:hidden;}

  1. HEAD_mdl-16940.patch
    28/Oct/09 9:35 AM
    9 kB
    Rossiani Wijaya
  2. MDL-16940.diff
    30/Jul/09 11:48 AM
    0.3 kB
    Jonathan Harker

Issue Links

Activity

Hide
Julien Boulen added a comment -

Hello,

We had same problem. This bug appears when we allow ajax in course ("disablecourseajax" variable).

This option add css classname "drag" in body tag.

In waiting better, we added .drag selectt{visibility: visible;} in ./theme/standard/styles_ie6.css

Cheers

Show
Julien Boulen added a comment - Hello, We had same problem. This bug appears when we allow ajax in course ("disablecourseajax" variable). This option add css classname "drag" in body tag. In waiting better, we added .drag selectt{visibility: visible;} in ./theme/standard/styles_ie6.css Cheers
Hide
Jonathan Harker added a comment - - edited

Julien's approach is correct - this issue is fixed by tweaking styles_ie6.css rather than YUI. Attached a patch.

Show
Jonathan Harker added a comment - - edited Julien's approach is correct - this issue is fixed by tweaking styles_ie6.css rather than YUI. Attached a patch.
Hide
Nicolas Connault added a comment -

Resolved, thanks for the fix!

Show
Nicolas Connault added a comment - Resolved, thanks for the fix!
Hide
Rossiani Wijaya added a comment -

The patch works for mine too. Thank you.

Resolved.

Closing

Show
Rossiani Wijaya added a comment - The patch works for mine too. Thank you. Resolved. Closing
Hide
Rossiani Wijaya added a comment -

Re-opeing the issue to fix the drop down disappearance on HEAD.

Patch HEAD_mdl-16940.patch is available to fix the drop down disappearance on HEAD.

Show
Rossiani Wijaya added a comment - Re-opeing the issue to fix the drop down disappearance on HEAD. Patch HEAD_mdl-16940.patch is available to fix the drop down disappearance on HEAD.
Hide
Martin Dougiamas added a comment -

We really had better not patch the YUI files. Can't we do this with an override style in theme/standard instead? (And also be a bit more specific in the class selector so that it refers to exactly what we want )

Show
Martin Dougiamas added a comment - We really had better not patch the YUI files. Can't we do this with an override style in theme/standard instead? (And also be a bit more specific in the class selector so that it refers to exactly what we want )

Dates

  • Created:
    Updated:
    Resolved: