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

Confusing 'See all' link when you have no notifications

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: 3.2
    • Component/s: Messages
    • Labels:
    • Testing Instructions:
      Hide
      1. Log into moodle with an account
      2. Open the notifications popover in the nav bar
      3. Click the "see all" link at the bottom of the popover
      4. Confirm you are taken to the dedicated notifications page (/message/output/popup/notifications.php) rather than just reloading the current page
      Show
      Log into moodle with an account Open the notifications popover in the nav bar Click the "see all" link at the bottom of the popover Confirm you are taken to the dedicated notifications page (/message/output/popup/notifications.php) rather than just reloading the current page
    • Affected Branches:
      MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE
    • Pull Master Branch:
      MDL-57075-master

      Description

      Steps to reproduce:

      1. Log in to https://qa.moodle.net/ as a student
      2. Click the Notifications icon
      3. Click the 'See all' link

      Expected result:

      There is no 'See all' link if you have no notifications.

      Actual result:

      The 'See all' link takes you to the page you are currently on i.e. it results in the Notifications window closing.

        Attachments

          Activity

            People

            Assignee:
            ryanwyllie Ryan Wyllie
            Reporter:
            tsala Helen Foster
            Peer reviewer:
            Mark Nelson
            Integrator:
            Dan Poltawski
            Tester:
            Adrian Greeve
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              5/Dec/16