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

          Attachments

            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