Download & Extend

Module system names should be on admin/build/modules, as well as UI names

Project:Drupal core
Version:8.x-dev
Component:system.module
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs work
Issue tags:Usability

Issue Summary

Use case: You just installed contrib module img_assist, nat, or i18n and are therefor most familiar with it by the name you just saw on the file when you downloaded it (or installed it with Drush).

You go to your modules page to enable this module for the first time, and faced with the huge list of modules, you fall back on ctrl+F to search for what you were just dealing with. There's a chance you won't find it because only the human-readable name for the module can differ a bit from the file name: Image assist, Node auto title, and Internationalization in this example, but contrib can get even more cryptic.

That's the basic use case but the point is that most people who will be using the module administration page will be handling the module files themselves at some point, one way or another. We shouldn't be hiding the module's system name for the sake of a slightly simpler interface.

A good spot for the system name would be right before or above the version number.

Comments

#1

Attached, find two patches.

The first (system-admin-inc.193034.patch) satisfies the request raised with this issue as it renames the module 'Name' column to 'Module Name', and adds a new column called 'System Name'. Further, 'System Name' is populated using the path of the module.

As feared, this solution does clutter the system module list, therefore I am including an alternative patch (system-admin-inc.193034-alt.patch) that leaves the columns as they were, however, hovering your mouse over the module 'Name' causes the module path to appear in a pop-up.

The alternative patch has the advantage of not adding clutter to the system modules page, though it does have the disadvantage not being searchable.

Note that the alternative patch is complemented by an update to system.module (system-module.193034-alt.patch) to provide help that reveals the otherwise hidden feature.

AttachmentSizeStatusTest resultOperations
system-admin-inc.193034.patch1.54 KBIgnored: Check issue status.NoneNone
system-admin-inc.193034-alt.patch1.45 KBIgnored: Check issue status.NoneNone
system-module.193034-alt.patch1.38 KBIgnored: Check issue status.NoneNone

#2

Status:active» needs review

#3

These patches all work. Thanks galapas! If people like what they do, they're ready to be committed.

I'm still holding out for, somewhere -- maybe right before or right after the description, so most of the time we wouldn't even be taking up any more space on the page, vertically or horizontally, having just the system name (without the path). The path, which now that I've seen it on the modules page thanks to galapas' patch, is awesome to have but could be a pop-up, since there's not really any call to be able to search for it.

I tried to make a patch that did this, just a variation of galapas', but I screwed it up somehow. I'll try again in a day, several deadlines to make first.

#4

Please note that the alternative patch (files system-admin-inc.193034-alt.patch and system-module.193034-alt.patch) do provide the pop-up already.

To be clear, the initial patch (file system-admin-inc.193034.patch) adds a 'System Name' column.

And the "*-alt.patch files provide the solution as a pop-up.

#5

Yes– and I think the best solution is the system name (without path) somewhere where it takes the least room, with the path as a pop-up ;-)

#6

Status:needs review» needs work

I agree with showing the help link only on admin/build/modules page. see: http://drupal.org/node/192962

So please move the module system name to the help page. It is as good as the tooltip solution.

#7

Status:needs work» needs review

but accept other reviews. did not mean to change the status

#8

Version:6.x-dev» 7.x-dev
Category:bug report» feature request

Sounds more like a feature request to me.

Robin

#9

Title:Useability: module system names should be on admin/build/modules, as well as UI names» Usability: module system names should be on admin/build/modules, as well as UI names

I don't like this. It's not useful. It makes the UI confusing.

#10

Component:system.module» usability

Moving all usability issues to Drupal - component usability.

#11

+1, I not only love this, I need this. System names of modules are very hard to find out without opening a shell, and finding out where the module is (the equivalent of "which" in Unix) is impossible without doing so - and looking at {system} if the module is installed in multiple places.

If four columns are too much clutter, consider removing the version column, which is only useful if you are neither using update status nor CVS to keep your modules updated.

#12

Please provide screenshot.

#13

One year after the last patch, the code has completely changed. I therefore rewrote the following variants:

  1. Adding a separate column for the module's filename
  2. Using a combined column for both the module's filename and the version
  3. Using a combined column for both the module's filename and the version + adding a hover with the complete path of the module

Enclosed are 3 patches and 3 screenshots (including 'as-is' but not the hover version).
Now this is really CNR...

AttachmentSizeStatusTest resultOperations
as-is.jpg84.69 KBIgnored: Check issue status.NoneNone
own-column.jpg97.79 KBIgnored: Check issue status.NoneNone
own-column.patch1.82 KBIdleFailed: Failed to apply patch.View details | Re-test
combined-column.jpg99.42 KBIgnored: Check issue status.NoneNone
combined-column.patch1.65 KBIdleFailed: 8484 passes, 1 fail, 0 exceptionsView details | Re-test
combined-hover.patch1.84 KBIdleUnable to apply patch combined-hover.patchView details | Re-test

#14

Status:needs review» reviewed & tested by the community

Fantastic, Pancho. Combined hover is awesome! I think it would be even better with the .module part of the module name removed (since this will always be there, it's not adding information), but even with that I'd call combined-hover.patch ready to be committed.

benjamin, Agaric Design Collective

#15

Status:reviewed & tested by the community» needs work

The last submitted patch failed testing.

#16

Status:needs work» needs review

re-testing.

#17

I prefer the combined view as well. I wonder if we should also use the small font size for the combined column? No real preference, just floating the idea.

#18

+1 for combined column.

Smaller font seems like a good idea too - after all, if its mainly there for search purposes then it doesn't have to be too big - and small means the page doesn't get quite as cluttered.

However i'm not sure we need the hover function if the system name is visible anyway though.

#19

Hover gives the path. Not needed for search, but a big bonus in letting you know exactly what module is in use (a module could be located in profiles/example/modules, sites/default/modules/, or sites/all/modules for instance).

#20

I am kind of disturbed by this patch, it seems to add a lot of clutter to a interface that is already very hard to scan trough. Calling for a smaller font basicly means, we want to make it less prominent (which I dont think is achieved by a smaller font). I see the case for people who have trouble finding the module, but does the occurrence of this problem weight up against the valuable screen real estate it takes up?

#21

The occurrence of the problem is frequent for anyone who builds or maintains a Drupal sites with many modules. Modules file names (which are your main reference for the module in the Drupal.org project path and when you download or use Drush) are equivalent to the system names, so it is key that they appear, and be searchable, on the listing page. Over the version works very well (echoing the tar file packaging) and makes good use of space.

Here's a small tweak to remove the ".module" file extension from the display. (Using substr– there's probably a better way, but at least you can see it, slightly de-cluttered.)

benjamin, Agaric Design Collective

AttachmentSizeStatusTest resultOperations
taxonomy-module-system-name-combined-hover-193034-21.patch.txt1.85 KBIgnored: Check issue status.NoneNone

#22

Can you please supply a screenshot with your change? I still think that this patch would add to much clutter, isn't there a better way to solve this problem (at its occurance, the naming in CVS?)

The problem only exists, when you install a module that has a conflicting map name versus actual module name. Where we would like to use this page for more purposes, and making it a easier starting point to set permissions and configure your module propperly (illustrated here : http://groups.drupal.org/node/10919#comment-38637 ) however if we add system names to this page, we have to carefully rethink for who and if we still can balance this page.

Can't we put more emphasis on the modules, that have just been uploaded?

#23

Status:needs review» needs work

The last submitted patch failed testing.

#24

Title:Usability: module system names should be on admin/build/modules, as well as UI names» Module system names should be on admin/build/modules, as well as UI names
Component:usability» system.module

#25

Here's a patch that does three things:

  1. Adds the module key to the version so you get "blog-7.0-dev"
  2. Moves the version column over to the right
  3. Makes the font smaller and lighter in color, and aligns right... Should I align the table header right too? looks a little of balance

I like this because it unclutters the module name and description, AND gives me the full package name, so I think it solves both problems described here.

I also just think it looks better. The specific version information isn't all that helpful except in specific situations. This makes just browsing and visually searching the modules page easier on my eyes.

AttachmentSizeStatusTest resultOperations
column-on-right.JPG92.15 KBIgnored: Check issue status.NoneNone
system.patch2.35 KBIgnored: Check issue status.NoneNone

#26

Screenshot looks good to me - would make it actually possible to find modules using CTRL-F in firefox instead of getting lost in the dependency sea. Patch needs some whitespace fixes though. Also I think it's probably OK too leave the $version variable rather than using $extra.

#27

I am still convinced that this patch adds a lot of clutter (making it gray, doesn't account for the 100 words + column more on this page), especially concerning all of the new ideas that are for this page that might be more in line with the workflow - we should really find a way to fix this without this method.

#28

Yeah, I wasn't sure about the $module['version'] variable... I added it to $extra because the $module array comes direct from module_rebuild_cache. I didn't want to mess with the $module['version'] var in case it was used somewhere else.

re: #27, I do understand that the modules page needs a total overhaul, I just thought this small modification would help start that.

#29

I like this idea best out of the ones proposed so far. Mainly because it doesn't add another column, and only mimimal text. Its definitely a step up from the current ui, even if there could be a better alternative still.

#30

@Magnity - This does add another column-- I still think this system name can:

1. appear under the name (small and gray, careernerd's look in that respect is fine by me)
2. have the system path in a title tool-tip on hover-over!

@Bojhan - this provides lots of information and usability with almost no extra space and minimal clutter. Usability win. (I also agree that core should have a page of recently added modules featuring configuration links, but that should be a separate issue.)

#31

First off, it doesn't add a column it just moves it.

Secondly, I think this page could use a multitude of enhancements.

Some information I think we should add:

  1. Multiple modules found: If there are multiple modules found in various directories, it should at the very least indicate that, and show which copy of the module is being used.
  2. Update status: If there is an update available it should show that here.
  3. Link to project page: There should be a link to the drupal.org/project page.
  4. Module Search: If the right Jquery is used, we could add an AHAH search to quickly filter the list down to what we're looking for. If this method is used, we could hide the version info altogether if we coded it to search that too. Note: I do understand that if we do this, we should provide the proper non-JS fallback code.

Of course, adding all of this extra information would clutter up the form. We should try to keep this page as simple and clean as possible if we are say we believe in making things easy for novices.

This issue is probably best merged with #363319: Redesign module page ...

I think any more work on this should be part of work towards a total redesign of the modules page.

#32

Added a screenshot of a mockup for a total overhaul of the modules page... on this ticket: #363319: Redesign module page

I think we should close this ticket and move all discussion to that ticket.

#33

Let's keep this issue for now. This is a small, focused usability win, whereas the other issue is a very big change.

#34

Version:7.x-dev» 8.x-dev

This is D8 now. I still really, really, really want this and think most people who use the modules page are 60% of the time enabling a module after getting it via a download or drush (and so see the project name in both cases) would really benefit from ctrl f here.

Adding system name search to module_filter module and trying to find a good way to put this in contrib are my D7 priorities.

Finding someone to hold Bojhan down while this gets committed is my D8 priority ;-)

#35

And, last mention until D8 dev begins... as a general principal, i don't think we should veto small wins on the basis that a much bigger win is necessary. That said, the disparity between the *projects* on the admin/reports/updates page versus the *modules* on admin/modules makes me think that lacking the project name, rather than the module system name, is the key usability issue. So enforcing grouping by project (the lowercase end of the d.o/project/namehere path, not any fickle changeable title), and turning package into a tagging/sorting option is a better fix...

#36

Anyway, I am not convinced this is necessary for a majority of our users and even only for a small subset - even webchick and merlinofchoas said it probably won't be above 10%. Polluting the module functionality screen with stuff that will be useful only occasionally, to me sounds silly - since it will have a big impact visually adding another element. I am not saying we shouldn't do small improvements, because there are larger fundamental issues - but I am saying, we should aim for a solution that has little to no impact visually - using a title attribute could work for example.

You can hold me down, but this is not a matter of opinion - its a a UX argument and you are not considering it.

nobody click here