Moodle
  1. Moodle
  2. MDL-37987

table headers swapped in course/manage.php

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Administration
    • Labels:

      Description

      In the latest 2.5 code, the two table headers "edit" and "move category to:" on the page course/manage.php in edit mode are swapped and therefore on the wrong columns.

      There seems to be a similar confusion with the classnames attached to the columns, "actions" and "icons", which I assume are both referring to the row of actions/icons, rather than to the dropdown select with move targets.

      Can I suggest while I'm here that "Move category to..." with an ellipsis might look better and make more sense than the current colon.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Rajesh Taneja added a comment -

            Thanks for reporting this David.

            I've put that on the backlog and assigned to Marina, as she has been working on this piece of code.

            In the meantime feel free to help us work on this issue. If you are able to provide a patch or links to your Git repository branch, please add a patch label so we will spot it.

            Show
            Rajesh Taneja added a comment - Thanks for reporting this David. I've put that on the backlog and assigned to Marina, as she has been working on this piece of code. In the meantime feel free to help us work on this issue. If you are able to provide a patch or links to your Git repository branch, please add a patch label so we will spot it.
            Hide
            Rajesh Taneja added a comment -

            Linking this to meta issue (MDL-31830) and adding Sam as watcher.

            Show
            Rajesh Taneja added a comment - Linking this to meta issue ( MDL-31830 ) and adding Sam as watcher.
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Integrated (master only), thanks!

            Show
            Eloy Lafuente (stronk7) added a comment - Integrated (master only), thanks!
            Hide
            Adrian Greeve added a comment -

            Tested on the master integration branch.

            The 'edit' and 'move category to' titles are now in the right order.

            Test passed.

            Show
            Adrian Greeve added a comment - Tested on the master integration branch. The 'edit' and 'move category to' titles are now in the right order. Test passed.
            Hide
            Damyon Wiese added a comment -

            Thanks for your hard work - this issue has made it! Moodle is now a little bit better.

            Regards, Damyon

            Show
            Damyon Wiese added a comment - Thanks for your hard work - this issue has made it! Moodle is now a little bit better. Regards, Damyon

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: