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. right.png
          right.png
          12 kB
        2. wrong.png
          wrong.png
          12 kB
        3. download.png
          download.png
          0.7 kB
        4. Screen Shot 2015-10-13 at 3.45.52 PM.png
          Screen Shot 2015-10-13 at 3.45.52 PM.png
          104 kB

          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