Page 1 of 1

Custom File Icons new switches

Posted: 04 Jun 2023 01:52
by jupe
I was testing out the new CFI switches, and noticed that in 24.40.0209 all lnk links to normal folders get the Access Denied icon when using /x.

Re: Custom File Icons new switches

Posted: 04 Jun 2023 02:47
by noembryo
Same thing here, too.
I must add that for the last couple of months, the .lnk files had the system's folder icon.
Previously they had the same custom icon that all the folders used in my system.
I can't remember the version that changed that though.. :cry:

Re: Custom File Icons new switches

Posted: 04 Jun 2023 03:00
by jupe
@noembryo: Yeah that was a requested and purposeful change, if you are talking about this:

Code: Select all

v24.10.0011 - 2023-02-25 09:28
    ! Custom File Icons: A specific LNK file icon assigned via Shell Context Menu 
      Properties was overwritten by the generic CFI file type icon. It shouldn't be. Fixed.
Maybe it should have only affected file, not folder links, although I can't see the issue you are describing.

Re: Custom File Icons new switches

Posted: 04 Jun 2023 03:22
by noembryo
jupe wrote: 04 Jun 2023 03:00 @noembryo: Yeah that was a requested and purposeful change, if you are talking about this:

Code: Select all

v24.10.0011 - 2023-02-25 09:28
    ! Custom File Icons: A specific LNK file icon assigned via Shell Context Menu 
      Properties was overwritten by the generic CFI file type icon. It shouldn't be. Fixed.
Maybe it should have only affected file, not folder links, although I can't see the issue you are describing.
I checked using a backup of 24.20.0300 and it has that problem (a minor annoyance perhaps)
Any folder.lnk file that I create has the default folder icon, although I use a CFI for open/closed folder..
With the current version, it has the "Access denied" icon, like @jupe reported.

Re: Custom File Icons new switches

Posted: 04 Jun 2023 15:36
by admin
Coincidentally, I saw both problems this morning while looking at something else before seeing this thread. Should be fixed in the next beta.

Re: Custom File Icons new switches

Posted: 04 Jun 2023 17:27
by noembryo
admin wrote: 04 Jun 2023 15:36 Coincidentally, I saw both problems this morning while looking at something else before seeing this thread. Should be fixed in the next beta.
Fixed. :tup:
Thank you.. :appl:

Re: Custom File Icons new switches

Posted: 05 Jun 2023 03:10
by jupe
Not sure if this matters, but while testing .0211, and I know it's only necessary to use one of the switches, but I noticed that if someone has 2 entries, one of each /f /e and no **\ entry then the curfolder in the tree ends up being the shell folder icon for normal folders.

Re: Custom File Icons new switches

Posted: 05 Jun 2023 07:55
by admin
Yes, I saw that too. Fix on the way.

Re: Custom File Icons new switches

Posted: 15 Jun 2023 04:19
by jupe
Historically when creating a menu or inputsel with a icon reference, I could just use an asterisk as shorthand (which I used extensively) for the default CFI folder icon, but now for inputsel (popupmenu ok) that always returns the Access Denied icon if a /x CFI entry exists. Now I do have the option of changing all my scripts over to use *.<DIR>, but that doesn't work in internal menus such as Favorites Folders captions anyway, whereas asterisk used to, so before I go about changing all the references, or decide to not use the /x CFI, I just thought I'd check if you think it is possible to restore the old behavior, which still works in popupmenu.

Code: Select all

  seticons("+*\ /x><xy>");
  $folders = regexreplace("<pickdirs * <xydata>>", "^(.+?)$", "$1||*.<DIR>");
  $folders = replace($folders, "\Panes||*.<DIR>", "\Panes||*");
  inputselect(, $folders, <crlf>, 1025);

Re: Custom File Icons new switches

Posted: 15 Jun 2023 07:42
by admin
Works better in next beta.