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

        1. MDL-32685.jpg
          42 kB
        2. myImage (1).png
          13 kB
        3. workdpress-plugins.png
          77 kB

          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 Mudrak 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 Mudrak 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 Mudrak 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 Mudrak 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 Mudrak added a comment -

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

            Show
            mudrd8mz David Mudrak added a comment - Attaching a screenshot from Wordpress Plugins page that was sort of inspiration for the current layout.
            Hide
            mudrd8mz David Mudrak 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 Mudrak 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: