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

flatpak icon support #1428

Merged
merged 2 commits into from
Nov 25, 2024
Merged

flatpak icon support #1428

merged 2 commits into from
Nov 25, 2024

Conversation

ohchase
Copy link
Contributor

@ohchase ohchase commented Nov 23, 2024

Addresses: #1333

  1. Flatpak has strict naming for any files, must begin with app id (exchange.haveno.Haveno)
  2. Icon for desktop set to exchange.haveno.Haveno as is standard
  3. icon.png renamed to exchange.haveno.Haveno.png
  4. AppDir haveno.svg moved to exchange.haveno.Haveno.svg (aligns desktop file icon prefix separate from extension)
  5. Update hard coded icon.png in package.gradle

Screenshot from 2024-11-23 14-45-39

@ohchase ohchase requested a review from woodser as a code owner November 23, 2024 19:49
@@ -35,7 +35,7 @@ modules:
- mkdir -p /app/share/icons/hicolor/128x128/apps/
- mkdir -p /app/share/applications/
- mkdir -p /app/share/metainfo/
- mv icon.png /app/share/icons/hicolor/128x128/apps/haveno.png
- mv exchange.haveno.Haveno.png /app/share/icons/hicolor/128x128/apps/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To avoid renaming icon.png, which is a dependency for other installers, does it make sense / is it possible to rename icon.png to exchange.haveno.Haveno.png here, and leave icon.png as is?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep this works, retained the icon.png naming in bb57f9e

@woodser woodser merged commit a541799 into haveno-dex:master Nov 25, 2024
6 checks passed
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.

2 participants