Moodle
  1. Moodle
  2. MDL-37907

participants list checkbox location inconsistent with other user listings

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4.1
    • Fix Version/s: 2.5
    • Component/s: Course
    • Labels:
    • Testing Instructions:
      Hide

      While logged in as a normal teacher or admin user
      Enter a course that contains users - view the partipants list (link via people block)
      make sure the "brief" participants layout is being shown.
      check to make sure the checkboxes appear as the first column in the table
      probably a good idea to try some bulk actions to ensure it's working.

      check the detailed participants layout and make sure the checkbox is in the same location as previously - (see screenshots attached for examples)

      Show
      While logged in as a normal teacher or admin user Enter a course that contains users - view the partipants list (link via people block) make sure the "brief" participants layout is being shown. check to make sure the checkboxes appear as the first column in the table probably a good idea to try some bulk actions to ensure it's working. check the detailed participants layout and make sure the checkbox is in the same location as previously - (see screenshots attached for examples)
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull Master Branch:
      master_MDL-37907

      Description

      the "brief" participants page layout places the checkbox as the last column of the table - most other reporting pages that list users in a table place the checkbox in the first column.

      This becomes a problem when the size of the middle column is small and in some themes (magazine is a good example) the checkbox is completely hidden unless using the scroll at the bottom of the table - but in using the scroll you can hide the users name and can't tell which user you're making the selection on.

      I think it makes sense to change the layout of the page to place the checkbox before the users name like on other report pages (quiz reports/assignment reports etc)

      I'm not sure about the "user details" view of the participants page - I think it makes sense to leave the checkbox in it's current location though.

        Gliffy Diagrams

          Activity

          Hide
          Dan Marsden added a comment -

          adding Helen as she may have some ideas here - I'm guessing she might use the participants page a bit too?

          Dan - if there's general agreement that we should do something here, feel free to assign it back to me to do the work

          Show
          Dan Marsden added a comment - adding Helen as she may have some ideas here - I'm guessing she might use the participants page a bit too? Dan - if there's general agreement that we should do something here, feel free to assign it back to me to do the work
          Hide
          Helen Foster added a comment -

          Hi Dan, thanks for adding me as a watcher. I'm attaching screenshots showing the checkboxes on the participants page (also one where they are hidden) and on the quiz grades and assignment grading pages.

          Looking at the screenshots I would agree that it would be better for the checkboxes to appear next to the user pictures on the participants page with user list brief.

          For the participants page with user details, the checkboxes are not hidden so easily, so I would also agree with NOT moving these checkboxes.

          Show
          Helen Foster added a comment - Hi Dan, thanks for adding me as a watcher. I'm attaching screenshots showing the checkboxes on the participants page (also one where they are hidden) and on the quiz grades and assignment grading pages. Looking at the screenshots I would agree that it would be better for the checkboxes to appear next to the user pictures on the participants page with user list brief. For the participants page with user details, the checkboxes are not hidden so easily, so I would also agree with NOT moving these checkboxes.
          Hide
          Dan Poltawski added a comment -

          Thanks for adding the screenshots Helen - that helps me see it and agree too!

          Show
          Dan Poltawski added a comment - Thanks for adding the screenshots Helen - that helps me see it and agree too!
          Hide
          Dan Poltawski added a comment -

          Do you think we need more consensus than us 3?

          Show
          Dan Poltawski added a comment - Do you think we need more consensus than us 3?
          Hide
          Dan Marsden added a comment -
          Show
          Dan Marsden added a comment - probably a good idea: https://moodle.org/mod/forum/discuss.php?d=221750
          Hide
          Dan Marsden added a comment -

          couple of more votes - I've put some basic patches up for review - will let integrators decide if this should go into stable releases or just master.

          Show
          Dan Marsden added a comment - couple of more votes - I've put some basic patches up for review - will let integrators decide if this should go into stable releases or just master.
          Hide
          Sam Hemelryk added a comment -

          Hi Dan, changes look great thank you. Feel free to push this to integration when you are ready.

          In regards to back-porting as this is purely a visual change I am ever so slightly leaning towards master only.

          Many thanks
          Sam

          Show
          Sam Hemelryk added a comment - Hi Dan, changes look great thank you. Feel free to push this to integration when you are ready. In regards to back-porting as this is purely a visual change I am ever so slightly leaning towards master only. Many thanks Sam
          Hide
          Dan Marsden added a comment -

          Thanks Sam - I think that makes sense - have updated master branch and removed others

          Show
          Dan Marsden added a comment - Thanks Sam - I think that makes sense - have updated master branch and removed others
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated (master only), thanks!

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

          Works as described. Passing.

          Show
          Andrew Davis added a comment - Works as described. Passing.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          This is valid for unlimited entries to the, soon to be unveiled, Moodle Codebase Gardens. It includes free access to all facilities.

          Personal and non-transferable to all assignees, reviewers and testers in this issue. Valid until switching to Blackboard (100000€ penalization will be applied).

          Thanks, closing as fixed!

          Show
          Eloy Lafuente (stronk7) added a comment - This is valid for unlimited entries to the, soon to be unveiled, Moodle Codebase Gardens. It includes free access to all facilities. Personal and non-transferable to all assignees, reviewers and testers in this issue. Valid until switching to Blackboard (100000€ penalization will be applied). Thanks, closing as fixed!
          Hide
          Mary Cooch added a comment -

          Removing docs_required label as I'm not sure this needs documenting as it's a change that makes sense anyway -but if anyone disagrees, please speak or feel free to add to the docs.

          Show
          Mary Cooch added a comment - Removing docs_required label as I'm not sure this needs documenting as it's a change that makes sense anyway -but if anyone disagrees, please speak or feel free to add to the docs.

            People

            • Votes:
              2 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: