KISS icon indicating copy to clipboard operation
KISS copied to clipboard

Icons are stuck as pebble shape

Open KSPAtlas opened this issue 3 years ago • 10 comments

Describe the bug A clear and concise description of what the bug is.

To Reproduce Steps to reproduce the behavior:

  1. Use Meeyo icon theme (squircle)
  2. See issue

Expected behavior Squircle shaped icons, should change when the setting is changed

Screenshots Screenshot_20211207-114111_1

Additional information

  • App version: 3.16.5
  • App installed via: (f-droid, Play Store, debug APK): F-Droid

Additional context Changing the icon shape via settings doesn't help

KSPAtlas avatar Dec 07 '21 11:12 KSPAtlas

Restart from Settings?

licaon-kter avatar Dec 07 '21 11:12 licaon-kter

Restart should work, but should not be necessary in my opinion. On change of settings favorites are simply not informed that something has changed: Thus icons are shown like before change.

Has anybody time to add a call of onFavoriteChange somehow on change of icons? https://github.com/Neamar/KISS/blob/53f8e88f76e584612e58bfcfab8c79ce2778b7b8/app/src/main/java/fr/neamar/kiss/IconsHandler.java#L80 https://github.com/Neamar/KISS/blob/e729c9db2d853268f1723881c4b30fa44841e6c9/app/src/main/java/fr/neamar/kiss/forwarder/Favorites.java#L120 I'm not sure what's the intended way of doing this, calling it directly or using broadcasts like e.g MainActivity.LOAD_OVER or MainActivity.FULL_LOAD_OVER

marunjar avatar Dec 07 '21 12:12 marunjar

Restarting from settings didn't help

KSPAtlas avatar Dec 07 '21 16:12 KSPAtlas

So the icons in the default pack respect the setting, bit the custom ones don't?

KSPAtlas avatar Dec 07 '21 23:12 KSPAtlas

I checked with another icon pack, and Meeyo is the only one that has this. Does Meeyo have something special about it?

KSPAtlas avatar Dec 07 '21 23:12 KSPAtlas

You'd need to report to them I guess.

licaon-kter avatar Dec 08 '21 04:12 licaon-kter

Maybe icon pack providers a mask which is then applied. @KSPAtlas can you check if you can change shape of contact icons? There is a toggle for applying icon pack mask.

marunjar avatar Dec 08 '21 06:12 marunjar

I can change contact icons, but also the icon pack isn't even pebble by default its meant to be squircle (which I want)

KSPAtlas avatar Dec 08 '21 07:12 KSPAtlas

I see, i'll try to fix this. I'd propose if shape is set to something else than "system default" this should simply override mask of icon pack.

But there's one caveat, if an icon from pack is smaller than chosen shape this this will lead to unwanted effects. Same if it's an iconpack with additional background and front image, if this is the case "system default" may be the only useful option.

marunjar avatar Dec 08 '21 10:12 marunjar

i tried to fix this with https://github.com/Neamar/KISS/pull/1846 so shape will be applied to all icons then. but i'm not sure if this will work out with meeyo icon pack because the icons have a size which fits in most other shapes you can use. For better understanding i have some screenshots, where the red background is for used shape:

  • circle Screenshot_1639233521
  • squircle Screenshot_1639233580
  • random teardrops Screenshot_1639233708

marunjar avatar Dec 11 '21 15:12 marunjar