Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-28363

Hidden courses in navigation are not dimmed.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.0.4, 2.1.1, 2.2
    • Component/s: Themes
    • Labels:
      None
    • Testing Instructions:
      Hide

      1/ select Binarius theme
      2/ Hide a course and expand tree in navigation block.

      Show
      1/ select Binarius theme 2/ Hide a course and expand tree in navigation block.
    • Workaround:
      Hide

      Dimmed link is in theme/base/style/core.css. The solution is to add missing css definition on line 27 in theme/base/style/core.css

      Missing css definition to add on line 27
      a.dimmed_text:link,
      a.dimmed_text:visited,

      Show
      Dimmed link is in theme/base/style/core.css. The solution is to add missing css definition on line 27 in theme/base/style/core.css Missing css definition to add on line 27 a.dimmed_text:link, a.dimmed_text:visited,
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w28_MDL-28363_m22_dimmed

      Description

      In navigation block hidden courses are not dimmed.

        Gliffy Diagrams

          Activity

          Hide
          skodak Petr Skoda added a comment -

          I had to add following too:
          .dimmed_text a:link,
          .dimmed_text a:visited,

          I suppose you have used some non-standard theme too, right?

          In any case more specific selectors should fix this and similar problems.

          Show
          skodak Petr Skoda added a comment - I had to add following too: .dimmed_text a:link, .dimmed_text a:visited, I suppose you have used some non-standard theme too, right? In any case more specific selectors should fix this and similar problems.
          Hide
          skodak Petr Skoda added a comment -

          Thanks for the report and proposed patch.

          Petr

          Show
          skodak Petr Skoda added a comment - Thanks for the report and proposed patch. Petr
          Hide
          petko81 Peter Svec added a comment -

          Thanks for integration...

          Show
          petko81 Peter Svec added a comment - Thanks for integration...
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Petr integrated now

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Petr integrated now
          Hide
          salvetore Michael de Raadt added a comment -

          Test result: passed. No worries.

          Show
          salvetore Michael de Raadt added a comment - Test result: passed. No worries.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                1/Aug/11