Moodle
  1. Moodle
  2. MDL-32685

Plugins overview: actions indent incosistency

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Minor 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
            Michael de Raadt added a comment -

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

            Show
            Michael de Raadt added a comment - Thanks for pointing that out. What would you suggest as an alternative?
            Hide
            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
            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
            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
            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
            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
            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
            Ruslan Kabalin added a comment -

            Added Andrew, he had an opinion I believe.

            Different colour might help actually.

            Show
            Ruslan Kabalin added a comment - Added Andrew, he had an opinion I believe. Different colour might help actually.
            Hide
            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
            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
            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
            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
            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
            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
            Andrew Nicols added a comment -

            Here's a quick mockup of what I mean

            Show
            Andrew Nicols added a comment - Here's a quick mockup of what I mean
            Hide
            David Mudrak added a comment -

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

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