unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by default
@ 2022-06-21 12:46 Stefan Kangas
  2022-06-21 19:44 ` Lars Ingebrigtsen
  2022-06-22 11:29 ` bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by, default David Ponce
  0 siblings, 2 replies; 6+ messages in thread
From: Stefan Kangas @ 2022-06-21 12:46 UTC (permalink / raw)
  To: 56123

[-- 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 --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by default
  2022-06-21 12:46 bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by default Stefan Kangas
@ 2022-06-21 19:44 ` Lars Ingebrigtsen
  2022-06-22  9:48   ` Stefan Kangas
  2022-06-22 11:29 ` bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by, default David Ponce
  1 sibling, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2022-06-21 19:44 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 56123

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





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by default
  2022-06-21 19:44 ` Lars Ingebrigtsen
@ 2022-06-22  9:48   ` Stefan Kangas
  0 siblings, 0 replies; 6+ messages in thread
From: Stefan Kangas @ 2022-06-22  9:48 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 56123

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).





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by, default
  2022-06-21 12:46 bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by default Stefan Kangas
  2022-06-21 19:44 ` Lars Ingebrigtsen
@ 2022-06-22 11:29 ` David Ponce
  2022-06-23 11:10   ` Stefan Kangas
  1 sibling, 1 reply; 6+ messages in thread
From: David Ponce @ 2022-06-22 11:29 UTC (permalink / raw)
  To: Stefan Kangas, Lars Ingebrigtsen; +Cc: 56123, 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







^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by, default
  2022-06-22 11:29 ` bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by, default David Ponce
@ 2022-06-23 11:10   ` Stefan Kangas
  2022-06-23 11:48     ` David Ponce
  0 siblings, 1 reply; 6+ messages in thread
From: Stefan Kangas @ 2022-06-23 11:10 UTC (permalink / raw)
  To: David Ponce; +Cc: 56123, Lars Ingebrigtsen

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!





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by, default
  2022-06-23 11:10   ` Stefan Kangas
@ 2022-06-23 11:48     ` David Ponce
  0 siblings, 0 replies; 6+ messages in thread
From: David Ponce @ 2022-06-23 11:48 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 56123, Lars Ingebrigtsen

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





^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-06-23 11:48 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-21 12:46 bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by default Stefan Kangas
2022-06-21 19:44 ` Lars Ingebrigtsen
2022-06-22  9:48   ` Stefan Kangas
2022-06-22 11:29 ` bug#56123: Set recentf-menu-filter to recentf-show-abbreviated by, default David Ponce
2022-06-23 11:10   ` Stefan Kangas
2022-06-23 11:48     ` David Ponce

Code repositories for project(s) associated with this inbox:

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).