Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Allow tray icon customization #576

Open
wants to merge 31 commits into
base: main
Choose a base branch
from
Open

Conversation

MrGarlic1
Copy link
Contributor

@MrGarlic1 MrGarlic1 commented May 4, 2024

Added option in Vesktop settings to select a custom tray icon, resolves #571

image

Only tested on Fedora 40 KDE. May need additional testing on Windows, I don't currently have access to a Windows device right now.

@Vendicated
Copy link
Member

Vendicated commented May 5, 2024

wow looks really good so far especially for a first contribution!

i'm not sure about the ui though, it would probably be much better to actually show the image that is chosen and maybe have it so you can click the image to change it? something like the profile picture picker (but way more simplistic)

also i can see a bit of inconsistent formatting, make sure you have prettier properly configured!

@MrGarlic1
Copy link
Contributor Author

MrGarlic1 commented May 5, 2024

I had some issues with displaying an image from the local filesystem on the Vesktop settings page. I ended up moving the tray icon setting to a tray icon context menu item, where I think it makes more sense to be. (old method kept on a branch in case this ends up being worse)

image

When selected, it brings up this window which has the same functionality as the setting did before while also showing the current icon. This could probably use some additional formatting to make it look better.

image

@Tiagoquix
Copy link
Contributor

Many thanks for your PR!

To be honest, I don't think an option as "important" as this should be so easily accessible with via right-click. Vendicated's idea ends up fitting better (of displaying the icon on the settings page next to the "Tray Icon" option, instead of your idea of adding new buttons).

@MrGarlic1
Copy link
Contributor Author

MrGarlic1 commented May 6, 2024

Many thanks for your PR!

To be honest, I don't think an option as "important" as this should be so easily accessible with via right-click. Vendicated's idea ends up fitting better (of displaying the icon on the settings page next to the "Tray Icon" option, instead of your idea of adding new buttons).

That's my end goal too, I just shied away due to literally never having used css or html. After much learning (read: pain), the recent commits moved it back to being in the Vesktop settings menu, with a better UI:

image

When hovered (test icon shown):

image

Currently the edit icon image is a hard link to a discord attachment which I don't think are permanent links anymore. Unsure if I should store the icon somewhere in the project itself or try and find a permanent URL for it.

@TwilightKiddy
Copy link

Played around with it under Arch with KDE Plasma, working mighty fine with the exception of a missing edit icon.
Is it possible to allow .svg and .ico files to be chosen? They are far more common for distributing app icons, as far as I'm aware.

@MrGarlic1
Copy link
Contributor Author

Played around with it under Arch with KDE Plasma, working mighty fine with the exception of a missing edit icon. Is it possible to allow .svg and .ico files to be chosen? They are far more common for distributing app icons, as far as I'm aware.

I tried doing this earlier, but Electron's NativeImage (which Tray uses) can only create images from .png and .jpg files. (RIP GIF tray icons)

@Tiagoquix
Copy link
Contributor

To the author of this PR: could you test if this also changes the icon when pressing alt+tab? (In KDE, there is the Vencord icon when you press alt+tab.)

@rootiest
Copy link

rootiest commented May 22, 2024

To the author of this PR: could you test if this also changes the icon when pressing alt+tab? (In KDE, there is the Vencord icon when you press alt+tab.)

On KDE I can override the icon used in the task switcher by:

  • First change the icon for Vesktop in your app menu to your preferred icon, then
  • Right-click on the Vesktop titlebar and go to More Actions > Configure Special Application Settings
  • Click the Detect Window Properties button
  • Select the Vesktop window with the crosshair
  • Click on the first one: Window Class Application
  • Click Add Property then Desktop Filename
  • Make sure this matches the name of the .desktop file used in your App menu, typically just "vesktop"
  • Choose "Apply Now"
  • Click Apply, and then verify the icon has changed to your liking
  • Change "Apply Now" to "Force"
  • Name and save the new rule.
  • Close and reopen Vesktop to verify the icon change sticks.
  • Enjoy your new icon in the App menu, taskbar, and app switcher (not tray, yet!)

Sorry this might seem kind of long/complicated, but it's actually pretty straightforward once you do it

@Tiagoquix
Copy link
Contributor

@rootiest Thank you very much! I knew KDE had these special properties, but I didn't know it reached this level of customization!

And, in fact, it's pretty straightforward to do this (just like you said).

Have a wonderful day, my friend!

@Covkie Covkie mentioned this pull request Jun 18, 2024
13 tasks
@SomeAspy
Copy link
Contributor

It might be worth adding an option to use the favicon the discord website sends, because this dynamically updates the number of pings for just about any number. We do this in armcord, if you would like to see a working example.
https://github.com/ArmCord/ArmCord/blob/573c96b4ba80705b1bc5f85c5a737258481023e9/src/discord/window.ts#L168

@quack-o
Copy link

quack-o commented Dec 20, 2024

awesome work, was looking for this, any progress on this

edit: seems like its already merged, why cant i see the feature in vesktop settings ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a new option to change the tray icon used by Vesktop
9 participants