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

Icon t/download.png should match t/download.svg

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Enable badges
      2. As a teacher in a course, create a new badge, set the criteria to manually award it and enable the badge
      3. Award the badge to a student
      4. Login as the stuednt
      5. Go to Preferences > Badges > Manage badges
      6. Inspect the Download icon next to the badge in an SVG capable browser, it should look nice when zoomed
      7. In your config.php set $CFG->svgicons = false;
      8. Refresh page
      9. Verify the icon matches the svg icon but is a png
      Show
      Enable badges As a teacher in a course, create a new badge, set the criteria to manually award it and enable the badge Award the badge to a student Login as the stuednt Go to Preferences > Badges > Manage badges Inspect the Download icon next to the badge in an SVG capable browser, it should look nice when zoomed In your config.php set $CFG->svgicons = false; Refresh page Verify the icon matches the svg icon but is a png
    • Affected Branches:
      MOODLE_25_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE, MOODLE_29_STABLE
    • Pull Master Branch:
      MDL-40442-master

      Description

      A new SVG icon has been introduced (MDL-35073), but its PNG fallback is quite different and not in the same style. A PNG should be created from that SVG icon.

      (And I guess the source could even added to the main Illustrator file).

      On a side note, the source of the SVG should be optimised, as the layers are not clean (greater file size, slower rendering?)

        Attachments

        1. download.png
          0.7 kB
          Barbara Ramiro
        2. right.png
          12 kB
          Barbara Ramiro
        3. Screen Shot 2015-10-13 at 3.45.52 PM.png
          104 kB
          Mark Nelson
        4. wrong.png
          12 kB
          Barbara Ramiro

          Issue Links

            Activity

              People

              • Assignee:
                barbararamiro Barbara Ramiro
                Reporter:
                fred Frédéric Massart
                Peer reviewer:
                Dan Poltawski
                Integrator:
                Andrew Nicols
                Tester:
                Mark Nelson
                Participants:
                Component watchers:
                Yuliya Bozhko, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, Bas Brands, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, Barbara Ramiro, Jake Dallimore, Jun Pataleta
              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  9/Nov/15