Add option to display module path on admin/modules table

Here's an updated patch for the 7.x-2.x branch.
Works as expected.

+++ b/module_filter.theme.inc
@@ -40,6 +40,12 @@ function theme_module_filter_system_modules_tabs($variables) {
+  $show_module_path = variable_get('module_filter_show_path', 0);
+  if ($show_module_path) {
+    $path = array(t('Path'));
+    array_splice($header, 3, 0, $path);
+  }
+
   if (variable_get('module_filter_track_recent_modules', 1)) {

This is unnecessary complicated code.

As the use case is who provides the module the path are probably not important. Why not just match on type like

        $module_path = drupal_get_path('module', $key);
        $types = array(
          'core' => 'modules',
          'profile' => 'profiles/',
          'all' => 'sites/all/modules',
          'site' => 'sites/'
        );

I'm not sure why the column width is so wide :-/

I'm not sure why the column width is so wide :-/

Because it had no styling.

This patch sets it to 10% with. And shows the whole path when hovering over the path column.

Just fixing a small typo issue on last patch

The patch is working fine to me! I can see the paths of my module.

Changing to RTBC

Evidence:

Guess this needs a D8 version first?

Hi clemens! Im trying to port this feature to d8. Wanna help? https://www.drupal.org/node/2886842

I'd like to keep the number of columns to a minimum. This patch moves the path into the description column and shows the entire relative path to the module. CSS will clip the path with an ellipsis when it's too long to fit in the column's width. Also moved the setting into the tabs fieldset as this only applies when using tabs.

I liked @greenSkin approach!

Here is the patch with some coding standards issue resolved.

I successfully tested the last patch and it's working fine.

In #2886842: Add option to display module path on admin/modules table I suggested to move it's patch into this issue and set version here into 8.x-3.x as both are same feature request. We have a backport patch in here for the maintainers to review later on. What do you think?

Tested. Works as expected.