Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.3
    • Fix Version/s: 2.4
    • Component/s: Usability
    • Labels:
    • Testing Instructions:
      Hide

      Test pre-requisites

      • An event of each type on calendar (course, group, user and site)

      Test steps

      1. Browse the calendar and make sure the icons appear nicely
      2. Also check the upcoming events block
        • Test RTL
        • Fallback on PNG icons
        • Consistency in padding/margin, etc...
      Show
      Test pre-requisites An event of each type on calendar (course, group, user and site) Test steps Browse the calendar and make sure the icons appear nicely Also check the upcoming events block Test RTL Fallback on PNG icons Consistency in padding/margin, etc...
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-36775-master

      Description

      Calendar icon issues

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

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

              Note: I have decide to deprecate the c/ directory. Only the calendar is using those icons which were copy of the icons under i/ (except for c/site). That's why I renamed them to i/fooevent. Also, please note that the courseevent icon is a placeholder and should be approved by Martin in MDL-36765.

              Show
              fred Frédéric Massart added a comment - Note: I have decide to deprecate the c/ directory. Only the calendar is using those icons which were copy of the icons under i/ (except for c/site). That's why I renamed them to i/fooevent. Also, please note that the courseevent icon is a placeholder and should be approved by Martin in MDL-36765 .
              Hide
              poltawski Dan Poltawski added a comment -

              Integrated, thanks guys

              Show
              poltawski Dan Poltawski added a comment - Integrated, thanks guys
              Hide
              markn Mark Nelson added a comment -

              Looks good Freddos in LTR and RTL for both the png and svg icons. Passing.

              Show
              markn Mark Nelson added a comment - Looks good Freddos in LTR and RTL for both the png and svg icons. Passing.
              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