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

Forum go to first unread post / mark thread read usability regression

XMLWordPrintable

    • MOODLE_31_STABLE, MOODLE_32_STABLE, MOODLE_33_STABLE
    • MOODLE_31_STABLE, MOODLE_32_STABLE
    • MDL-58055-master
    • Hide

      In clean theme:

      1. As admin, create a course, enrol a student, add a forum, make a post.
      2. As student, ensure unread forum post tracking is on.
      3. Go to the forum, on the page where it lists the discussions
        1. Confirm that there is no padding on the "Mark all as read" link on the side of the link to view
      Show
      In clean theme: As admin, create a course, enrol a student, add a forum, make a post. As student, ensure unread forum post tracking is on. Go to the forum, on the page where it lists the discussions Confirm that there is no padding on the "Mark all as read" link on the side of the link to view

      Ages ago, I implemented MDL-32359, which was a small, but quite important usability win.

      Then tonight, many time on Moodle.org, I found that I had marked a forum thread read, when I was trying to click the link to go to the first unread post.

      I think this was broken by MDL-57093, which did not in itself introduce the stupid padding: 0 rule that is to blame. That came from MDL-42711. The problem with MDL-57093 is that it changed the selectors to be less specific, so the wrong style rule in MDL-42711 suddenly took effect.

      Anyway, there is now an annoying usability bug. Please can someone fix it.

            dobedobedoh Andrew Lyons
            timhunt Tim Hunt
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Ankit Agarwal Ankit Agarwal
            Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.