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

block_myoverview: Remove icons from course context menu (Was: Starred courses should get a "unstar" icon in their context menu)

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.6
    • 3.8
    • Dashboard (My home)
    • MOODLE_36_STABLE
    • MOODLE_38_STABLE
    • MDL-64743-master
    • Hide

      Verify removed icons

      • Login as a user who is enrolled in at least one course
      • Go to the Dashboard and look at your course overview block
      • Open a course's context menu
      • Use the "Star this course" / "Unstar this course" / "Hide this course" / "Show this course" menu items to toggle all possible status permutations.

      Expected result:
      The context menu items are never prefixed by any icon

      Show
      Verify removed icons Login as a user who is enrolled in at least one course Go to the Dashboard and look at your course overview block Open a course's context menu Use the "Star this course" / "Unstar this course" / "Hide this course" / "Show this course" menu items to toggle all possible status permutations. Expected result: The context menu items are never prefixed by any icon

    Description

      In the course overview block, courses can be starred which is a great feature.
      If a course is not starred yet, in the context menu of the course, there is an entry "star this course" which has a star icon.
      If a course is already starred, in the context menu of the course, there is an entry "unstar this course" which hasn't an icon. This is inconsistent.

      I would propose to add a the fa-star-o icon to the "unstar this course" menu item.

      Attachments

        Activity

          People

            abias Alexander Bias
            abias Alexander Bias
            Simey Lameze Simey Lameze
            Adrian Greeve Adrian Greeve
            Gladys Basiana Gladys Basiana
            David Woloszyn, Huong Nguyen, Jake Dallimore, Michael Hawkins, Stevani Andolo
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              18/Nov/19

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 30 minutes
                30m