Uploaded image for project: '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
    • Status: Closed
    • Priority: 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

          Attachments

            Activity

            Hide
            daniss 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
            daniss 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
            lazydaisy Mary Evans added a comment -

            Starting peer review

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

            +1 from me - looks good.

            Setting this for Integration review.

            Ciao
            Mary

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

            Thanks Daniele this has been integrated now

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

            looks ok to me, thanks

            Show
            skodak Petr Skoda added a comment - looks ok to me, thanks
            Hide
            damyon 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 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:
                  Fix Release Date:
                  18/Nov/13