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

Plugins overview only shows SVG icon if a non-SVG icon by the same name exists

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.2.2, 3.3
    • 3.1.6, 3.2.3
    • Administration
    • MOODLE_32_STABLE, MOODLE_33_STABLE
    • MOODLE_31_STABLE, MOODLE_32_STABLE
    • mdl-58485
    • Hide

      1. Visit admin/plugins.php and confirm that mod_forum has an icon (Bonus step: Look at the used image; confirm that it contains SVG XML tags).
      2. Delete or rename the icon mod/forum/pix/icon.png on your filesystem, leaving the SVG in place.
      3. Visit admin/plugins.php again; confirm that mod_forum still has an icon.

      Show
      1. Visit admin/plugins.php and confirm that mod_forum  has an icon (Bonus step: Look at the used image; confirm that it contains SVG XML tags). 2. Delete or rename the icon mod/forum/pix/icon.png on your filesystem, leaving the SVG in place. 3. Visit admin/plugins.php again; confirm that mod_forum  still has an icon.

    Description

      admin/plugins.php illustrates each plugin with its own icon. If an SVG icon is present, the SVG icon is preferred over PNG or other pixel formats.

      However, if the plugin only offers an SVG icon, no icon is displayed at all.

      Steps to reproduce:
      1. Visit admin/plugins.php and confirm that mod_forum has an icon (Bonus step: Look at the used image; confirm that it contains SVG XML tags).
      2. Delete or rename the icon mod/forum/pix/icon.png on your filesystem, leaving the SVG in place.
      3. Visit admin/plugins.php again; confirm that only a spacer is shown instead of the mod_forum icon (empty image with icon pluginicon noicon CSS classes.

      By the way, the activity module overview admin/modules.php respects the SVG icon as a standalone

      Attachments

        Issue Links

          Activity

            People

              jan.dagefoerde Jan Dageförde
              jan.dagefoerde Jan Dageförde
              Ankit Agarwal Ankit Agarwal
              David Monllaó David Monllaó
              David Mudrák (@mudrd8mz) David Mudrák (@mudrd8mz)
              Andrew Lyons, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              3 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                8/May/17