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

should calendar UI static elements be dynamic

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.7.3, 3.8, 3.9
    • Fix Version/s: None
    • Component/s: Calendar, Usability
    • Labels:
    • Affected Branches:
      MOODLE_37_STABLE, MOODLE_38_STABLE, MOODLE_39_STABLE

      Description

      This was considered when reviewing MDL-67300.

      Whenever the calendar page is shown, the month/upcoming... views are dynamically loaded. Same happens when you change / filter... that's nice.

      But there are some elements in the UI that never are refreshed, unless you reload the page. The more obvious are the buttons ("new event", "subscriptions" and "export").

      Because of that, we have very complex permissions evaluation code that looks for perms sitewide. Also, it's not very intuitive sometimes (if perms change for a given context / course), some of the buttons should become unavailable / dimmed. And they just don't behave that way now.

      So this is an issue about consider when those currently "static" elements could be dynamically calculated based on current filters. That way surely:

      1) We should not calculate perms site-wide, but just for the context/filter enabled.
      2) The UI would dynamically enable/disable elements that are not available, making the calendar page UI 100% consistent (not only some parts).

      This requires:

      1) An analysis about the impact of this (new WS calls, more fragments maybe, changes on permissions evaluation, ideally to a lighter approach...)
      2) Implement the new behaviors if agreed, completely covered.

      Note this may have some relation / dependency with MDL-67422 where the calendar permissions evaluation is being cleaned up.

      Ciao

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Unassigned
              Reporter:
              stronk7 Eloy Lafuente (stronk7)
              Participants:
              Component watchers:
              Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Barbara Ramiro, Bas Brands
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Dates

                Created:
                Updated: