terminal icon indicating copy to clipboard operation
terminal copied to clipboard

profile-icons with web sources are not shown on the taskbar

Open torgeros opened this issue 4 years ago • 10 comments

Windows Terminal version (or Windows build number)

1.8.1521.0

Other Software

No response

Steps to reproduce

I am using WINDOWS 11, I do not know how this behaves in Win10.

  1. Create a new profile (or use an existing one)
  2. For the icon, paste a weblink to an ico file (eg https://github.com/favicon.ico)
  3. As you will see, that works fine (the icon gets shown in new tabs and in the settings menu) image
  4. keep winterm open
  5. right-click onto the program in the taskbar
  6. As you will see, the icons are not rendered here image

Expected Behavior

Profile Icons are rendered the same way everywhere across the device

Actual Behavior

Profile Icons from the web are not rendered at all in the taskbar dialog

torgeros avatar Jul 03 '21 14:07 torgeros

You know, I am confident that we have a dupe for this floating around on the repo, but I can't find it. The same thing happens when using an emoji as the icon for a profile.

I suppose theoretically, we could actually download the image locally and set the path for the jumplist entry to that file.

zadjii-msft avatar Jul 06 '21 15:07 zadjii-msft

I suppose theoretically, we could actually download the image locally and set the path for the jumplist entry to that file.

I someone is going to implement that, maybe also make copies of local images to an app-internal path. I found it somewhat annoying that I have to make sure that my custom local icons do not get deleted by accident.

Like for example when I download an image, set it as profile icon, and then clean up my downloads folder, the profile icon is gone.

torgeros avatar Jul 07 '21 14:07 torgeros

Yeah -- we should cache local and remote paths, honestly. We need to be extra careful to not cache old copies of local icons that have changed, but that should present little issue.

Technically -- and unrelatedly, mind you -- we should also be caching them in memory. Reloading every image every time the settings changes is a right pain. :smile:

DHowett avatar Jul 07 '21 16:07 DHowett

Hello @torgeros, is this issue is still open, I like to work on this issue. I am new to open source and want to contribute.

yash112-lang avatar Jul 11 '21 13:07 yash112-lang

I am not planning on working on this myself. No one else has posted something about them working on it.

Feel free to fork the repo and then open a pull request referencing this issue.

torgeros avatar Jul 11 '21 16:07 torgeros

I've noticed this too - shows up fine within Terminal but not in the jump menu for my WSL instance... image image image

EDIflyer avatar Oct 23 '23 21:10 EDIflyer

This is still an issue

JavierCCC avatar May 27 '24 19:05 JavierCCC

This is still an issue

@JavierCCC try to re-pin it or reinstall.

N0xFF avatar May 31 '24 05:05 N0xFF

facing same issue

tejasness avatar Jun 26 '24 06:06 tejasness

This is still an issue

@JavierCCC try to re-pin it or reinstall.

didnt work

tejasness avatar Jun 26 '24 06:06 tejasness

Facing this issue too Image

yarkonev avatar Nov 10 '24 10:11 yarkonev

Soon, profile icons with web sources will not be shown at all. It's a trivial security risk.

DHowett avatar Nov 12 '24 17:11 DHowett

Soon, profile icons with web sources will not be shown at all. It's a trivial security risk.

Is that related to something with wsl --install, --import, or --export? Because it can't figure out the correct directory for shortcut.ico. Since version 2.4.x, I have to manually set the path for the profile icon of the distro that came from a WSL install.

Jesus Christ Windows Terminal....

atiliosantosuu avatar Mar 17 '25 14:03 atiliosantosuu

It's the same thing.

Image

dr-jagulator avatar Apr 01 '25 06:04 dr-jagulator