Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Badges, Usability
    • Labels:
    • Rank:
      49741

      Description

      The icon for badges should be monochrome.

      1. badge.svg
        1 kB
        Barbara Ramiro
      1. badge.png
        0.4 kB
      2. bonfire-screenshot-20130416-085222-139.png
        16 kB

        Issue Links

          Activity

          Hide
          Frédéric Massart added a comment -

          Added Barbara here.

          Show
          Frédéric Massart added a comment - Added Barbara here.
          Hide
          Barbara Ramiro added a comment -

          Here's the guidelines for moodle icons http://docs.moodle.org/dev/Moodle_icons

          Show
          Barbara Ramiro added a comment - Here's the guidelines for moodle icons http://docs.moodle.org/dev/Moodle_icons
          Hide
          Martin Dougiamas added a comment -

          Barbara can you just make one quickly?

          Show
          Martin Dougiamas added a comment - Barbara can you just make one quickly?
          Hide
          Martin Dougiamas added a comment -

          There is already a trophy icon made under t/award that would fine to replace i/badge

          Show
          Martin Dougiamas added a comment - There is already a trophy icon made under t/award that would fine to replace i/badge
          Hide
          Barbara Ramiro added a comment -

          Martin, the t/award looks more like a keyhole/mic/goblet and it needs resizing to 16x16 so i made a new trophy for i/badge instead.

          Here goes badge.png

          Let me know what you think

          Show
          Barbara Ramiro added a comment - Martin, the t/award looks more like a keyhole/mic/goblet and it needs resizing to 16x16 so i made a new trophy for i/badge instead. Here goes badge.png Let me know what you think
          Hide
          Martin Dougiamas added a comment -

          Sensational +1

          Show
          Martin Dougiamas added a comment - Sensational +1
          Hide
          Barbara Ramiro added a comment -

          Great! Submitted straight for integration.

          Show
          Barbara Ramiro added a comment - Great! Submitted straight for integration.
          Hide
          Damyon Wiese added a comment -

          Thanks Barbara, this is now integrated to master. I bumped the version (to clear the cache) and ran "theme/base/cli/svgtool.php --ie9fix" on this in integration.

          Show
          Damyon Wiese added a comment - Thanks Barbara, this is now integrated to master. I bumped the version (to clear the cache) and ran "theme/base/cli/svgtool.php --ie9fix" on this in integration.
          Hide
          Damyon Wiese added a comment -

          Just testing that the new icon is displayed after upgrading.

          Show
          Damyon Wiese added a comment - Just testing that the new icon is displayed after upgrading.
          Hide
          Damyon Wiese added a comment -

          It works!

          Show
          Damyon Wiese added a comment - It works!
          Hide
          Barbara Ramiro added a comment -

          Yay thanks Damyon!

          Show
          Barbara Ramiro added a comment - Yay thanks Damyon!
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Did you think this day was not going to arrive ever?

          Your patience has been rewarded, yay, sent upstream, thanks!

          Closing...ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Did you think this day was not going to arrive ever? Your patience has been rewarded, yay, sent upstream, thanks! Closing...ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: