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

Colours alternation is missing from tables in formal_white (and in most of other themes)

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.4, 2.2.1, 2.3
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: Themes
    • Labels:
      None

      Description

      Table's rows should have alternate background colors for accessibility reasons.
      Formal white is missing them as it can easily seen in the user account list.
      As usual, I attach the code to patch formal_white code but, IMHO, this patch should be applied at central level (base or canvas) and not in a peripheral theme.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              daniss Daniele Cordella created issue -
              daniss Daniele Cordella made changes -
              Field Original Value New Value
              Assignee Patrick Malley [ ptrkmkl ] Daniele Cordella [ daniss ]
              daniss Daniele Cordella made changes -
              Status Open [ 1 ] Waiting for integration review [ 10010 ]
              Pull Master Diff URL https://github.com/kordan/moodle/compare/master...MDL-31712_master
              Pull Master Branch MDL-31712_master
              Pull from Repository git://github.com/kordan/moodle.git
              Fix Version/s 2.3 [ 10657 ]
              daniss Daniele Cordella made changes -
              Summary themeth Colours alternation is missing from tables in formal_white (and in most of other themes)
              daniss Daniele Cordella made changes -
              Description Table's rows should have alternate background colors for accessibility reasons.
              Formal white is missing them as it can easily seen in the user account list.
              As usual, I attach the code to patch formal_white but IMHO this patch should be applied at central level (base or canvas) and not in a peripheral theme.
              Table's rows should have alternate background colors for accessibility reasons.
              Formal white is missing them as it can easily seen in the user account list.
              As usual, I attach the code to patch formal_white code but, IMHO, this patch should be applied at central level (base or canvas) and not in a peripheral theme.
              daniss Daniele Cordella made changes -
              Fix Version/s STABLE backlog [ 10463 ]
              Fix Version/s 2.3 [ 10657 ]
              stronk7 Eloy Lafuente (stronk7) made changes -
              Currently in integration Yes [ 10041 ]
              nebgor Aparup Banerjee made changes -
              Status Waiting for integration review [ 10010 ] Integration review in progress [ 10004 ]
              Integrator nebgor
              daniss Daniele Cordella made changes -
              Attachment copy.gif [ 27154 ]
              stronk7 Eloy Lafuente (stronk7) made changes -
              Testing Instructions go to Site administration > Users > Accounts > Browse list of users and look at the list.
              Row's background colors should alternate.
              # enable formal_white theme in the site.
              # go to Site administration > Users > Accounts > Browse list of users and look at the list.
              # Row's background colors should alternate.
              stronk7 Eloy Lafuente (stronk7) made changes -
              Integrator nebgor stronk7
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Integration review in progress [ 10004 ] Waiting for testing [ 10005 ]
              Affects Version/s 2.2.1 [ 11456 ]
              Affects Version/s 2.1.4 [ 11452 ]
              Fix Version/s 2.1.5 [ 11553 ]
              Fix Version/s 2.2.2 [ 11552 ]
              Fix Version/s STABLE backlog [ 10463 ]
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Waiting for testing [ 10005 ] Testing in progress [ 10011 ]
              Tester stronk7
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Testing in progress [ 10011 ] Tested [ 10006 ]
              stronk7 Eloy Lafuente (stronk7) made changes -
              Link This issue has a non-specific relationship to MDL-31847 [ MDL-31847 ]
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Tested [ 10006 ] Closed [ 6 ]
              Resolution Fixed [ 1 ]
              Currently in integration Yes [ 10041 ]
              Integration date 09/Mar/12

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    12/Mar/12