Moodle

Unread icon for forums

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.3.3
  • Fix Version/s: 2.4
  • Component/s: Usability
  • Labels:
  • Testing Instructions:
    Hide

    Test pre-requisites

    • Enable the unread post tracking
    • For the student edit their profile and set 'Forum tracking' to 'Yes: highlight new posts for me'
    • Create some posts using a different account on a forum

    Test steps

    1. Login and visit the forum's page (and the forum/index.php page)
    2. Make sure the icon look nice and the fallback to PNG works as expected
    Show
    Test pre-requisites Enable the unread post tracking For the student edit their profile and set 'Forum tracking' to 'Yes: highlight new posts for me' Create some posts using a different account on a forum Test steps Login and visit the forum's page (and the forum/index.php page) Make sure the icon look nice and the fallback to PNG works as expected
  • Affected Branches:
    MOODLE_23_STABLE
  • Fixed Branches:
    MOODLE_24_STABLE
  • Pull from Repository:
  • Pull Master Branch:
    MDL-36782-master
  • Pull Master Diff URL:

Description

Unread icon for forums

t/clear, should be 12x12. Could be renamed to something that makes more sense, such as 'markasread'.

Issue Links

blocks

Sub-task - Issues are sometimes broken into multiple sub-tasks. MDL-36802 LTI settings icon issues

  • Minor - Minor loss of function where workaround is possible
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Dan Poltawski added a comment -

Hi Fred,

This is conflicting on LTI.

Show
Dan Poltawski added a comment - Hi Fred, This is conflicting on LTI.
Hide
Permalink
Frédéric Massart added a comment -

Hi Dan, conflict fixed. Thanks!

Show
Frédéric Massart added a comment - Hi Dan, conflict fixed. Thanks!
Hide
Permalink
Dan Poltawski added a comment -

Integrated, thanks Fred.

Show
Dan Poltawski added a comment - Integrated, thanks Fred.
Hide
Permalink
Mark Nelson added a comment -

Hi Fred, there was no spacing after 'Unread' and the tick.

Show
Mark Nelson added a comment - Hi Fred, there was no spacing after 'Unread' and the tick.
Hide
Permalink
Frédéric Massart added a comment -

Thanks Mark, pushed a new commit to fix that missing margin. Cheers!

Show
Frédéric Massart added a comment - Thanks Mark, pushed a new commit to fix that missing margin. Cheers!
Hide
Permalink
Dan Poltawski added a comment -

pulled.

Show
Dan Poltawski added a comment - pulled.
Hide
Permalink
Mark Nelson added a comment -

Great work Fred! Simply amazing, superb, fantastic, mesmerising. Such awesome work. I feel like crying. Passing.

Show
Mark Nelson added a comment - Great work Fred! Simply amazing, superb, fantastic, mesmerising. Such awesome work. I feel like crying. Passing.
Hide
Permalink
Frédéric Massart added a comment -

Thank Barbara

Show
Frédéric Massart added a comment - Thank Barbara
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

Y E S !

Closing as fixed, many thanks!

Show
Eloy Lafuente (stronk7) added a comment - Y E S ! Closing as fixed, many thanks!

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: