Details

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

      Description

      Anomaly theme uses serif type fonts by default. This is not recommended and needs to use the recommended sans-serif fonts as set by YUI CSS Fonts in the body tag of all themes by default.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              samhemelryk Sam Hemelryk added a comment -

              The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

              TIA and ciao

              Show
              samhemelryk Sam Hemelryk added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
              Hide
              lazydaisy Mary Evans added a comment -

              Rebased...

              Show
              lazydaisy Mary Evans added a comment - Rebased...
              Hide
              poltawski Dan Poltawski added a comment -

              Hi Mary,

              I have integrated this into master only as its the sort of change which would be too disruptive for users in the table branches.

              But noting the linked issue we need to have more discussion and consensus before deciding whether all themes should use sans serif fonts.

              There may be a place for diversity in fonts in themes and really this needs to be explored before deciding we are going to get rid of serifs in all themes.

              cheers,
              dan

              Show
              poltawski Dan Poltawski added a comment - Hi Mary, I have integrated this into master only as its the sort of change which would be too disruptive for users in the table branches. But noting the linked issue we need to have more discussion and consensus before deciding whether all themes should use sans serif fonts. There may be a place for diversity in fonts in themes and really this needs to be explored before deciding we are going to get rid of serifs in all themes. cheers, dan
              Hide
              nebgor Aparup Banerjee added a comment -

              I think Dan meant 'stable branches' above.

              Show
              nebgor Aparup Banerjee added a comment - I think Dan meant 'stable branches' above.
              Hide
              ankit_frenz Ankit Agarwal added a comment -

              test passed.
              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - test passed. Thanks
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              And this has landed upstream, finally! Yay!

              תודה רבה && شكرا جزيلا



              Closing, ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - And this has landed upstream, finally! Yay! תודה רבה && شكرا جزيلا Closing, ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    25/Jun/12