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:
    • Rank:
      47764

      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.

        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: