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

New "highlight forum post when deep linked" feature conflicts with "forum tracking" feature

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Log in
      2. Enable forum tracking in user preferences ("Yes, highlight...")
      3. Go to a forum where you've unread messages
      4. Click on the number (of unread posts) in the "Unread" column (NOT on the discussion title)
      5. The first unread post should be highlighted, or, alternatively, no highlight at all should be there.
      6. Observe that the blue line for highlighting the linked post is not above the first unread post like in the screenshot
      Show
      Log in Enable forum tracking in user preferences ("Yes, highlight...") Go to a forum where you've unread messages Click on the number (of unread posts) in the "Unread" column (NOT on the discussion title) The first unread post should be highlighted, or, alternatively, no highlight at all should be there. Observe that the blue line for highlighting the linked post is not above the first unread post like in the screenshot
    • Affected Branches:
      MOODLE_31_STABLE
    • Fixed Branches:
      MOODLE_31_STABLE
    • Pull Master Branch:
      MDL-54675-forum-highlight

      Description

      Steps to reproduce :

      1. Log in
      2. Enable forum tracking in user preferences ("Yes, highlight...")
      3. Go to a forum where you've unread messages
      4. Click on the number (of unread posts) in the "Unread" column (NOT on the discussion title)

      Actual result (bug):

      Observe that the blue line for highlighting the linked post is just above the first unread post (see screenshot).

      Expected result:

      The first unread post should be highlighted, or, alternatively, no highlight at all should be there.

      Comment: related to new feature MDL-53050 (slipped the QA test, unfortunately )

        Attachments

        1. MDL-53050.png
          MDL-53050.png
          30 kB
        2. screenshot-1.png
          screenshot-1.png
          18 kB
        3. screenshot-2.png
          screenshot-2.png
          18 kB
        4. screenshot-3.png
          screenshot-3.png
          13 kB
        5. screenshot-4.png
          screenshot-4.png
          61 kB

          Issue Links

            Activity

              People

              Assignee:
              brendanheywood Brendan Heywood
              Reporter:
              mina Nicolas Martignoni
              Peer reviewer:
              Andrew Nicols
              Integrator:
              David Monllaó
              Tester:
              John Okely
              Participants:
              Component watchers:
              Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                11/Jul/16