thetwom / Tuner

Tuner app
GNU General Public License v3.0
170 stars 16 forks source link

Add new instrument icons #81

Closed ahangarha closed 3 months ago

ahangarha commented 3 months ago

Hi

Thanks for the app. This is my primary tool for tuning my instruments.

I wonder if adding new (Persian) instrument icons to the app is possible. I am not an Android developer, but I can see some icons in XML format that I am unfamiliar with. They seem to be very similar to SVG files. Would it work if I shared SVG files with you?

I wanted to see how I can help with making new icons for you (if you would add them to the app)

thetwom commented 3 months ago

Thanks for the interest in improving the app. I guess a few more icons could be nice, so I would be glad for ideas.

I think, it would be important though to follow a few guidelines:

And yes, svg is perfect. See the attached file for an example. ic-guitar

ahangarha commented 3 months ago

Please check out these icons:

tar setar kamancheh
thetwom commented 3 months ago

Thanks a lot for the icons. To me they look as the fit good enough to the current style (I don't say that they perfectly fit, since I never had a strong style so there is not perfect fit :-) ). I hope that the degree of detail is not too high, but I guess, this should be better experienced within the app.

Could you give me the names of these instruments, then I can at get the naming right (its more about file names and descriptions).

Are you planning to provide more icons? If yes, can you give a rough number?

For the further way forward, I am proposing the following:

  1. We agree on the rough number of icons
  2. You provide the most important ones
  3. I integrate it in the app and provide a test version
  4. We check if there should be improvements and improve it if necessary
  5. Integrating more icons if wanted ...
ahangarha commented 3 months ago

I used alt attribute on each image. From left to right: Tar, Setar, Kamancheh.

I can make more but for my use, these are enough for the time being.

If you need me to make more icons, just let me know. I try to create them in a reasonable time.

thetwom commented 3 months ago

Ok, sounds good, will try to include these within the next few days.

thetwom commented 3 months ago

Here you can try it: v6.8.0-rc1. I simplified the icons slightly, but for the small size, this shouldn't be really visible. Android might complain during installation, since this is signed with my own keys (and not fdroid or google). In this case you would have to uninstall the app before. But make sure, you save predefined instrument lists if you want to keep them. And note, that the settings will be lost during removal.

ahangarha commented 3 months ago

Thanks for incorporating the new icons in the new release. I just made a typo fix in #83.

thetwom commented 3 months ago

Thanks, I created the release v6.8.0 it is now queued to appear on fdroid and google play. Should appear there within the next days.

ahangarha commented 3 months ago

Thanks for the smooth process. Bests