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
    • Rank:
      39638

      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.

      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: