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

Plugins overview: actions indent incosistency

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.2.2
    • Fix Version/s: DEV backlog
    • Component/s: Administration
    • Labels:
    • Affected Branches:
      MOODLE_22_STABLE

      Description

      My first concern when I opened the updated plugins overview (introduced in MDL-20438) was that if a plugin has no settings, the 'Uninstall' will be right below 'Settings' of the plugin above. This does not look quite user friendly IMHO.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              salvetore Michael de Raadt added a comment -

              Thanks for pointing that out. What would you suggest as an alternative?

              Show
              salvetore Michael de Raadt added a comment - Thanks for pointing that out. What would you suggest as an alternative?
              Hide
              kabalin Ruslan Kabalin added a comment -

              What would you suggest as an alternative?

              A separate table cells for Settings and Uninstall, so that each will be in own column (like it was really).

              Show
              kabalin Ruslan Kabalin added a comment - What would you suggest as an alternative? A separate table cells for Settings and Uninstall, so that each will be in own column (like it was really).
              Hide
              mudrd8mz David Mudrák added a comment -

              I intentionally merged these columns in 2.3 because the table was becoming too wide. When there is an update available for a plugin, we need to display that info at its row. And then there was just too much space needed. So what I did is that all actions like Setting, Uninstall etc are now displayed in one cell.

              I personally did not consider this as usability issue. Wordpress admin UI, for example, uses similar layout for action links a lot at their management screen.

              Let's see what others think.

              Show
              mudrd8mz David Mudrák added a comment - I intentionally merged these columns in 2.3 because the table was becoming too wide. When there is an update available for a plugin, we need to display that info at its row. And then there was just too much space needed. So what I did is that all actions like Setting, Uninstall etc are now displayed in one cell. I personally did not consider this as usability issue. Wordpress admin UI, for example, uses similar layout for action links a lot at their management screen. Let's see what others think.
              Hide
              mudrd8mz David Mudrák added a comment -

              Would it help if action links were displayed in different colour? Eg Uninstall in red, Settings in the default blue.

              Show
              mudrd8mz David Mudrák added a comment - Would it help if action links were displayed in different colour? Eg Uninstall in red, Settings in the default blue.
              Hide
              kabalin Ruslan Kabalin added a comment -

              Added Andrew, he had an opinion I believe.

              Different colour might help actually.

              Show
              kabalin Ruslan Kabalin added a comment - Added Andrew, he had an opinion I believe. Different colour might help actually.
              Hide
              sbourget Stephen Bourget added a comment -

              How about switching the order so that uninstall is listed before settings. This could potentially make it much more readable. The only module that would look out of place would be the forum module since it does not have an uninstall option.

              Show
              sbourget Stephen Bourget added a comment - How about switching the order so that uninstall is listed before settings. This could potentially make it much more readable. The only module that would look out of place would be the forum module since it does not have an uninstall option.
              Hide
              dobedobedoh Andrew Nicols added a comment -

              Stephen: Some plugins (e.g. all Auth plugins) have Settings but not Uninstall so this fix wouldn't work.

              Show
              dobedobedoh Andrew Nicols added a comment - Stephen: Some plugins (e.g. all Auth plugins) have Settings but not Uninstall so this fix wouldn't work.
              Hide
              dobedobedoh Andrew Nicols added a comment -

              WRT UX, consistency is key really. If at all possible, we should try and ensure that the Uninstall and Settings links line up all of the way down.

              Colours may help, but not for everyone (e.g. colour blind users).
              An Icon also may help – perhaps entirely replacing the text so that it's possible to use the spacer, but the current iconset isn't very clear and this wouldn't be consistent with other similar admin pages.
              Another alternative could be drop the Source and Version columns entirely and show those underneath the plugin name...

              Show
              dobedobedoh Andrew Nicols added a comment - WRT UX, consistency is key really. If at all possible, we should try and ensure that the Uninstall and Settings links line up all of the way down. Colours may help, but not for everyone (e.g. colour blind users). An Icon also may help – perhaps entirely replacing the text so that it's possible to use the spacer, but the current iconset isn't very clear and this wouldn't be consistent with other similar admin pages. Another alternative could be drop the Source and Version columns entirely and show those underneath the plugin name...
              Hide
              dobedobedoh Andrew Nicols added a comment -

              Here's a quick mockup of what I mean

              Show
              dobedobedoh Andrew Nicols added a comment - Here's a quick mockup of what I mean
              Hide
              mudrd8mz David Mudrák added a comment -

              Attaching a screenshot from Wordpress Plugins page that was sort of inspiration for the current layout.

              Show
              mudrd8mz David Mudrák added a comment - Attaching a screenshot from Wordpress Plugins page that was sort of inspiration for the current layout.
              Hide
              mudrd8mz David Mudrák added a comment -

              See the screenshot attached to MDL-20438 on why we need much more space in the last column (which was the reason for merging the columns into one).

              Show
              mudrd8mz David Mudrák added a comment - See the screenshot attached to MDL-20438 on why we need much more space in the last column (which was the reason for merging the columns into one).
              Hide
              rajeshtaneja Rajesh Taneja added a comment -

              Hello. I will not be able to work on this issue in the immediate future. In order to create a truer sense of the state of this issue and to allow other developers to have chance to become involved, I am removing myself as the assignee of this issue.

              For more information, see http://docs.moodle.org/dev/Changes_to_issue_assignment

              If you have any information about this issue or a possible fix please post it here

              Show
              rajeshtaneja Rajesh Taneja added a comment - Hello. I will not be able to work on this issue in the immediate future. In order to create a truer sense of the state of this issue and to allow other developers to have chance to become involved, I am removing myself as the assignee of this issue. For more information, see http://docs.moodle.org/dev/Changes_to_issue_assignment If you have any information about this issue or a possible fix please post it here

                People

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

                  Dates

                  • Created:
                    Updated: