Moodle
  1. Moodle
  2. MDL-39694 META: Colour trend improvements to Formal White theme
  3. MDL-40262

Improve the contrast of text within the breadcrumb for trends in formal_white.

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.6
    • Component/s: Themes
    • Labels:

      Description

      As identified by Fred in MDL-39626 the contrast of text both regular and link could be improved for items on the navbar.

        Gliffy Diagrams

          Activity

          Hide
          Daniele Cordella added a comment -

          I simply assigned black (that is well contrasted according to http://snook.ca/technical/colour_contrast/colour.html) to links and white (that makes a not compliant contrast with background color) to plain texts in the breadcrumb. This is the less expensive solution I found to differentiate plain text and links and have links well contrasted.

          Show
          Daniele Cordella added a comment - I simply assigned black (that is well contrasted according to http://snook.ca/technical/colour_contrast/colour.html ) to links and white (that makes a not compliant contrast with background color) to plain texts in the breadcrumb. This is the less expensive solution I found to differentiate plain text and links and have links well contrasted.
          Hide
          Mary Evans added a comment -

          Starting peer review

          Show
          Mary Evans added a comment - Starting peer review
          Hide
          Mary Evans added a comment -

          +1 from me - looks good.

          Setting this for Integration review.

          Ciao
          Mary

          Show
          Mary Evans added a comment - +1 from me - looks good. Setting this for Integration review. Ciao Mary
          Hide
          Sam Hemelryk added a comment -

          Thanks Daniele this has been integrated now

          Show
          Sam Hemelryk added a comment - Thanks Daniele this has been integrated now
          Hide
          Petr Skoda added a comment -

          looks ok to me, thanks

          Show
          Petr Skoda added a comment - looks ok to me, thanks
          Hide
          Damyon Wiese added a comment -

          This issue along with 77 of it's friends has been sent upstream and released to the world.

          Thankyou for your contribution.

          Show
          Damyon Wiese added a comment - This issue along with 77 of it's friends has been sent upstream and released to the world. Thankyou for your contribution.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: