mirror of
https://github.com/jpochyla/psst.git
synced 2026-04-27 15:35:56 +03:00
[PR #79] [CLOSED] Icon #421
Labels
No labels
api
bug
build
documentation
duplicate
enhancement
good first issue
help wanted
idea
invalid
linux
lowprio
macos
pull-request
upstream
windows
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/psst#421
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/jpochyla/psst/pull/79
Author: @Kethku
Created: 7/17/2021
Status: ❌ Closed
Base:
master← Head:icon📝 Commits (2)
4d038bedon't show console on windows machines when running release buildsb472758create icon and add build script to add it to windows executable📊 Changes
6 files changed (+68 additions, -1 deletions)
View changed files
📝
Cargo.lock(+10 -0)📝
psst-gui/Cargo.toml(+5 -1)➕
psst-gui/assets/logo.ico(+0 -0)➕
psst-gui/assets/logo.svg(+44 -0)➕
psst-gui/build.rs(+8 -0)📝
psst-gui/src/main.rs(+1 -0)📄 Description
Addresses part of https://github.com/jpochyla/psst/issues/74
Turns out druid doesn't yet have support for setting the window icon https://github.com/linebender/druid/issues/1162 so this PR just creates an icon from the logo svg path already present in this app and the spotify colors. It also introduces a build script in order to set the icon for the app exe on windows. Doing similar for other platforms will require more work that I don't know how to do.
Here is the icon I added:

This is obviously just a first pass, feel free to take it or leave it. Just figured I'd get the ball rolling.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.