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

      Test pre-requisites

      • A course event in a calendar
      • Combo list and course list on front page (front page settings)
      • Mnet set up to access a remote course

      Test steps

      1. Explore course/index.php and course/category.php
      2. Explore the course event on the calendar
      3. Check the 'Courses' block for the 'Course' and 'Remote course' icons
      4. Make sure:
        • The icons appear nicely
        • The fallback on PNG works (Opera, IE8)
      Show
      Test pre-requisites A course event in a calendar Combo list and course list on front page (front page settings) Mnet set up to access a remote course Test steps Explore course/index.php and course/category.php Explore the course event on the calendar Check the 'Courses' block for the 'Course' and 'Remote course' icons Make sure : The icons appear nicely The fallback on PNG works (Opera, IE8)
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-36765-master

      Description

      Opening a separate issue for those as it appears that Martin was not convinced by the proposed course icon.

      i/courseevent should also be updated.

      This issue could include the revised scale icon.

        Gliffy Diagrams

          Issue Links

            Activity

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

            Those icons can be found in the 'Courses' block. But the course icon can also be found on the front page.

            Show
            fred Frédéric Massart added a comment - Those icons can be found in the 'Courses' block. But the course icon can also be found on the front page.
            Hide
            fred Frédéric Massart added a comment -

            Note: The scale icon is not related but has been inserted in this issue by lazyness.

            Show
            fred Frédéric Massart added a comment - Note: The scale icon is not related but has been inserted in this issue by lazyness.
            Hide
            phalacee Jason Fowler added a comment - - edited

            [ ] Syntax
            [ ] Output
            [ ] Whitespace
            [ ] Language
            [ ] Databases
            [ ] Testing
            [ ] Security
            [ ] Documentation
            [ ] Git
            [ ] Sanity check
            [ ] Images

            Good to go!

            Show
            phalacee Jason Fowler added a comment - - edited [ ] Syntax [ ] Output [ ] Whitespace [ ] Language [ ] Databases [ ] Testing [ ] Security [ ] Documentation [ ] Git [ ] Sanity check [ ] Images Good to go!
            Hide
            nebgor Aparup Banerjee added a comment -

            thanks, thats been integrated into master.

            Show
            nebgor Aparup Banerjee added a comment - thanks, thats been integrated into master.
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            Works as described.
            Remote course icon is cool somewhat reminds me of steam

            cheers!

            Show
            ankit_frenz Ankit Agarwal added a comment - Works as described. Remote course icon is cool somewhat reminds me of steam cheers!
            Hide
            matteo Matteo Scaramuccia added a comment -

            n00b question (don't hit me ): https://github.com/FMCorz/moodle/compare/97df015690...MDL-36765-master#L3R9 seems to miss a run of theme/base/cli/svgtool.php and the same in integration.git: something that will be addressed automatically when the commits will land into moodle.git?

            Show
            matteo Matteo Scaramuccia added a comment - n00b question (don't hit me ): https://github.com/FMCorz/moodle/compare/97df015690...MDL-36765-master#L3R9 seems to miss a run of theme/base/cli/svgtool.php and the same in integration.git : something that will be addressed automatically when the commits will land into moodle.git ?
            Hide
            fred Frédéric Massart added a comment -

            Hi Matteo, we decided to run that script at the same time as we roll the releases. The tag will be placed in there don't worry . Thanks for your concern.

            Show
            fred Frédéric Massart added a comment - Hi Matteo, we decided to run that script at the same time as we roll the releases. The tag will be placed in there don't worry . Thanks for your concern.
            Hide
            matteo Matteo Scaramuccia added a comment -

            OK
            TNX Frédéric for taking the time to reply, appreciated!

            Show
            matteo Matteo Scaramuccia added a comment - OK TNX Frédéric for taking the time to reply, appreciated!
            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:
                1 Start watching this issue

                Dates

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