Moodle
  1. Moodle
  2. MDL-28363

Hidden courses in navigation are not dimmed.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor 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
    • Rank:
      18011

      Description

      In navigation block hidden courses are not dimmed.

        Activity

        Hide
        Petr Škoda 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
        Petr Škoda 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
        Petr Škoda added a comment -

        Thanks for the report and proposed patch.

        Petr

        Show
        Petr Škoda added a comment - Thanks for the report and proposed patch. Petr
        Hide
        Peter Svec added a comment -

        Thanks for integration...

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

        Thanks Petr integrated now

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

        Test result: passed. No worries.

        Show
        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: