Details

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

      Description

      The icon for badges should be monochrome.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              fred Frédéric Massart added a comment -

              Added Barbara here.

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

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

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

              Barbara can you just make one quickly?

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

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

              Show
              dougiamas Martin Dougiamas added a comment - There is already a trophy icon made under t/award that would fine to replace i/badge
              Hide
              barbararamiro 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
              barbararamiro 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
              dougiamas Martin Dougiamas added a comment -

              Sensational +1

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

              Great! Submitted straight for integration.

              Show
              barbararamiro Barbara Ramiro added a comment - Great! Submitted straight for integration.
              Hide
              damyon 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 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 Damyon Wiese added a comment -

              Just testing that the new icon is displayed after upgrading.

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

              It works!

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

              Yay thanks Damyon!

              Show
              barbararamiro Barbara Ramiro added a comment - Yay thanks Damyon!
              Hide
              stronk7 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
              stronk7 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:
                    Fix Release Date:
                    14/May/13