bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by default

bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by default

[Stefan Kangas]

[-- Attachment #1: Type: text/plain, Size: 452 bytes --]

Severity: wishlist

I just pushed commit 32906819ad to master, which adds
'recentf-show-abbreviated' as a possible value for
'recentf-menu-filter'.  This basically just makes the menu use e.g.
"~/foo" instead of "/home/skangas/foo".

I think this would be a strictly better default than what we have now,
so I suggest we change it.  I'm happy to flip the switch and update
the NEWS entry accordingly.

Please see the attached before/after screenshots.

[-- Attachment #2: before.png --]
[-- Type: image/png, Size: 144930 bytes --]

[-- Attachment #3: after.png --]
[-- Type: image/png, Size: 130633 bytes --]

bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by default

[Lars Ingebrigtsen]

Stefan Kangas <stefan@marxist.se> writes:

> I just pushed commit 32906819ad to master, which adds
> 'recentf-show-abbreviated' as a possible value for
> 'recentf-menu-filter'.  This basically just makes the menu use e.g.
> "~/foo" instead of "/home/skangas/foo".
>
> I think this would be a strictly better default than what we have now,
> so I suggest we change it.  I'm happy to flip the switch and update
> the NEWS entry accordingly.
>
> Please see the attached before/after screenshots.

Yes, looks much better.  Please go ahead and flip the switch.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by default

[Stefan Kangas]

close 56123 29.1
thanks

Lars Ingebrigtsen <larsi@gnus.org> writes:

> Yes, looks much better.  Please go ahead and flip the switch.

Thanks, now done (commit ec9228eb81).

bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by, default

[David Ponce]

Hello Stefan, Lars,

Regarding the last changes done to recentf.el (commit 32906819ad to 
master, which adds 'recentf-show-abbreviated' as a possible value for 
'recentf-menu-filter', I am not sure this change is actually needed. It 
could even be confusing with the option `recentf-filename-handlers', 
that can be used to achieve the same at a lower level.

You can customize `recentf-filename-handlers' to add handler
'abbreviate-file-name' in order to get abbreviated file names in the 
recentf menu (in my local setup I also added `file-true-name').

Make be it would be enough to change the default value of
`recentf-filename-handlers' ?

Just wanted to let you know ;-)

Thanks!
David

bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by, default

[Stefan Kangas]

David Ponce <da_vid@orange.fr> writes:

> Make be it would be enough to change the default value of
> `recentf-filename-handlers' ?

I think you are correct, so I have reverted the relevant changes and
changed that default instead.  I completely overlooked this somehow.
Thanks!

bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by, default

[David Ponce]

On 23/06/2022 13:10, Stefan Kangas wrote:
> David Ponce <da_vid@orange.fr> writes:
> 
>> Make be it would be enough to change the default value of
>> `recentf-filename-handlers' ?
> 
> I think you are correct, so I have reverted the relevant changes and
> changed that default instead.  I completely overlooked this somehow.
> Thanks!

You are welcome.
Thanks to you for your work on this library :-)

David