Motivation

When working with different directories that have some common files that have long full path name, it's hard to find the right file from the list. This is because the panel's width is 500 and the full path may not be shown completely within the panel as below

<-------- visible to  user --------> <----- invisible to users ------------------->

/-----------------------------------\
| /foo/bar/directory/with/very/long-|/path1/name/sample.txt
| Files -> Recent Files -> /foo/bar/|directory/with/very/long-/path1/name/sample.txt
| /foo/bar/directory/with/very/long-|/path2/name/sample.txt
| Files -> Recent Files -> /foo/bar/|directory/with/very/long-/path2/name/sample.txt
\-----------------------------------/

In this illustration, when looking up sample.txt, the user has no idea about the directory information (both path1 and path2 do matching)

The patches

• The panel's width is one half of the width of the Geany window when this amount is greater than or equals to 250
• Otherwise the panel's width defaults to the width of the Geany window
• For labels, don't display full names; only display the parent directory's basename and the file name

The previous illustration now becomes

/-----------------------------------\
| name1/sample.txt                  |
| Files -> Recent Files -> /foo/bar/|directory/with/very/long-/path/name1/sample.txt
| name2/sample.txt                  |
| Files -> Recent Files -> /foo/bar/|directory/with/very/long-/path/name2/sample.txt
\-----------------------------------/

and this helps user to quickly find the right files.

Of course this doesn't solve all edge cases when there are a few more levels of duplication in full path names.

You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany-plugins/pull/1394

Commit Summary

• ed4c564 commander: extend default panel width to 1k
• a96cfa6 commander: display less path information in dropbox menu
• f4548ec commander: set panel width according to main windows width

File Changes

(1 file)

• M commander/src/commander-plugin.c (201)

Patch Links:

• https://github.com/geany/geany-plugins/pull/1394.patch
• https://github.com/geany/geany-plugins/pull/1394.diff

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany-plugins/pull/1394@github.com>