Moodle
  1. Moodle
  2. MDL-29742

Extra class added when item hidden with Ajax enabled

    Details

    • Testing Instructions:
      Hide

      If you include the css below in a theme. You can see that the area surrounding a resource goes grey when it is hidden. If you reload the page whilst some resources are hidden the area will not be grey until you have toggled them visible and then hidden again

      .region-content .dimmed_text {
      background: none repeat scroll 0 0 #EDEDED;
      }

      Show
      If you include the css below in a theme. You can see that the area surrounding a resource goes grey when it is hidden. If you reload the page whilst some resources are hidden the area will not be grey until you have toggled them visible and then hidden again .region-content .dimmed_text { background: none repeat scroll 0 0 #EDEDED; }
    • Workaround:
      Hide

      In /course/lib.php function print_section

      just before

      echo html_writer::start_tag('div', array('class'=>join(' ', $classes)));

      place

      if ($accessiblebutdim){$classes[] = 'dimmed_text';}

      Show
      In /course/lib.php function print_section just before echo html_writer::start_tag('div', array('class'=>join(' ', $classes))); place if ($accessiblebutdim){$classes[] = 'dimmed_text';}
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Rank:
      19259

      Description

      When Ajax is enabled and a resource is hidden the class dimmed_text is appended to the div surrounding a resource. This is not set when Ajax disabled or on the initial page loading.

      As it can be styled it would be useful to have it available from page load

      e.g.
      .region-content .dimmed_text {
      background: none repeat scroll 0 0 #EDEDED;
      }

        Issue Links

          Activity

          Hide
          Matthew Cannings added a comment -

          Just checked on 2.3dev and it no longer inserts the dimmed_text class in the div surrounding the element but rather on the link to the resource itself.

          This CSS hack/problem is no longer relevant

          Show
          Matthew Cannings added a comment - Just checked on 2.3dev and it no longer inserts the dimmed_text class in the div surrounding the element but rather on the link to the resource itself. This CSS hack/problem is no longer relevant
          Hide
          Mary Evans added a comment -

          Does this mean we can close this?

          Show
          Mary Evans added a comment - Does this mean we can close this?
          Hide
          Matthew Cannings added a comment - - edited

          I guess so as a fix would not have any affect on 2.3 as it is at the moment and people could hack this into <2.2 if they wanted it. I was going to close it but not able to.

          Show
          Matthew Cannings added a comment - - edited I guess so as a fix would not have any affect on 2.3 as it is at the moment and people could hack this into <2.2 if they wanted it. I was going to close it but not able to.
          Hide
          Dan Poltawski added a comment -

          Thanks for clarifying, closing.

          Show
          Dan Poltawski added a comment - Thanks for clarifying, closing.

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: