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

table headers swapped in course/manage.php

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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

          Attachments

            Issue Links

              Activity

              Hide
              rajeshtaneja 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
              rajeshtaneja 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
              rajeshtaneja Rajesh Taneja added a comment -

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

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

              Integrated (master only), thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (master only), thanks!
              Hide
              abgreeve 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
              abgreeve 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 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 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:
                    Fix Release Date:
                    14/May/13