Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-32309

calendar event hide/show links are different on page and in block

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Test pre-requisites

      • To be checked on all themes
      • A course with a group
      • A calendar with 4 events
        • Group event called 'Group event'
        • Course event called 'Course event'
        • User event called 'User event'
        • Site event called 'Global event'
      • A calendar block in a course page

      Test steps

      1. Go to your course and visit the calendar
      2. Using the block Events key, hide/show each event type at a time
      3. Make sure the block actions work as expected
      4. Go back on the course index
      5. Make sure the styles in the calendar block look good
      Show
      Test pre-requisites To be checked on all themes A course with a group A calendar with 4 events Group event called 'Group event' Course event called 'Course event' User event called 'User event' Site event called 'Global event' A calendar block in a course page Test steps Go to your course and visit the calendar Using the block Events key , hide/show each event type at a time Make sure the block actions work as expected Go back on the course index Make sure the styles in the calendar block look good
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-32309-master

      Description

      While fixing MDL-30904, it was noticed by Sam:
      when viewing the calendar there is the Events Key fake block that gets added to the display and serves the sole purpose of hiding/showing event types.
      It does pretty much the same thing, but takes a different approach in how it structures the HTML.
      Perhaps it will be nice to have a standard approach (eye icon one), and accessibility should be considered too. Also, use of table is horrible for usability and should be replaced with a tableless solution (keeping accessibility and usability in mind).

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              fred Frédéric Massart
              Reporter:
              rajeshtaneja Rajesh Taneja
              Peer reviewer:
              Adrian Greeve
              Integrator:
              Dan Poltawski
              Tester:
              Ankit Agarwal
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

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