Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: Usability
    • Labels:
      None
    • Testing Instructions:
      Hide
      • To be tested with supported browsers and IE8 (or Opera) for PNG fallback.
      • Note: both icons are the same, but have different file names.
      1. Check the icon 'Assign roles' in blocks when editing mode
      2. Check the icon 'Assign roles' in the navigation block (Course admin > Users > Enrolled users)
      3. Check the icon 'Enrol users' in the navigation block (Course admin > Users > Other users)
      4. Check the icon 'Enrol users' in the course list page (Home ► Courses ► Miscellaneous ► Add/edit courses)
      Show
      To be tested with supported browsers and IE8 (or Opera) for PNG fallback. Note: both icons are the same, but have different file names. Check the icon 'Assign roles' in blocks when editing mode Check the icon 'Assign roles' in the navigation block (Course admin > Users > Enrolled users) Check the icon 'Enrol users' in the navigation block (Course admin > Users > Other users) Check the icon 'Enrol users' in the course list page (Home ► Courses ► Miscellaneous ► Add/edit courses)
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-36869-master

      Description

      Revised icons enrolusers and assignroles

        Gliffy Diagrams

          Attachments

            Activity

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

            Pushing straight for integration as straight forward and approved by Martin.

            Show
            fred Frédéric Massart added a comment - Pushing straight for integration as straight forward and approved by Martin.
            Hide
            nebgor Aparup Banerjee added a comment -

            this has been integrated.

            note about preserve tags being removed here from Fred:

            Oh that's Dan's fault, he ran the script on the SVG icons, but then Barbara had to redesign them
            Hence the tag which was removed
            The tag will be put back on release

            Show
            nebgor Aparup Banerjee added a comment - this has been integrated. note about preserve tags being removed here from Fred: Oh that's Dan's fault, he ran the script on the SVG icons, but then Barbara had to redesign them Hence the tag which was removed The tag will be put back on release
            Hide
            fred Frédéric Massart added a comment -

            Yeah well, I didn't mean "Dan's fault". What I meant to say was that Dan had run the script, and the icon had to be updated afterwards. Sorry :-/.

            Show
            fred Frédéric Massart added a comment - Yeah well, I didn't mean "Dan's fault". What I meant to say was that Dan had run the script, and the icon had to be updated afterwards. Sorry :-/.
            Hide
            phalacee Jason Fowler added a comment -

            Works fine in IE8 and Firefox 17

            Show
            phalacee Jason Fowler added a comment - Works fine in IE8 and Firefox 17
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Y E S !

            Closing as fixed, many thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Y E S ! Closing as fixed, many thanks!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  3/Dec/12