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

Icon pack: Folder icon always turns white #336

Open
Npepperlinux opened this issue Aug 3, 2024 · 2 comments · May be fixed by #344
Open

Icon pack: Folder icon always turns white #336

Npepperlinux opened this issue Aug 3, 2024 · 2 comments · May be fixed by #344
Labels
bug Something is not working as expected

Comments

@Npepperlinux
Copy link
Contributor

All folder icons are monochromatic white when the icon pack corresponding to Discreet Launcher is applied (Arcticons are white line icons so I can't find this problem).
I think it is related to the color filter setting for each folder icon.

Arcticons Arcticon White(black-line) Arcticons Material You Delta
discreet-launcher_with_arcticons discreet-launcher_with_arction-black discreet-launcher_with_arcticons-material-you discreet-launcher_with_delta
@falzonv
Copy link
Owner

falzonv commented Aug 9, 2024

Hello,

Thank you for the report and screenshots!
It is in fact related to the recent PR #322 and I am not sure how to manage this.

Each Discreet Launcher folder has a user-defined color applied on its base icon and app count (by default this is white, it can be edited by clicking the folder icon in the "Folders" menu entry).
So regardless of the icon color in the icon pack, the final result will be of the color defined in Discreet Launcher (white by default, but it could be any #RRGGBB and also supports transparency).

For the second screenshot you can fix it by changing the color for the "settings" folder to black.
@Donnnno: on the 2 first screenshots, it seems the base icon is a bit too high for where the app count appears, could you adjust it?

For the last 2 screenshots it's a bit more complicated, you can define a more fitting color (that will look better with other icons around) but since there is no "hole" in the icon for letting the app count appear it will be blend with the base icon.
@Donnnno, @GitGitro:
Would it be possible to insert a transparent part in these base icons so that the app count could appear?
I am not sure how to compute the position where the app count would appear in the icon XML, it is meant to be integrated in the |__| shape at the bottom of the Discreet Launcher icon that you can see in the screenshot below (this |__| shape is large enough to fit a 2-digit number with the same gap size with the sides and bottom lines).

Sorry guys for the inconveniences, I never thought when implementing folders 3 years ago that someday we would try to integrate them with icon packs...

Best regards

@falzonv falzonv added the bug Something is not working as expected label Aug 15, 2024
@kaanelloed
Copy link
Contributor

That's on me, I didn't explain the reasoning behind my commit and didn't follow-up.

This is why I originally wanted the icon pack to manage numbers by creating icons for each of them. This way, the launcher doesn't have to manage the colors, and the number is consistent with the style of the rest of the icon.

@kaanelloed kaanelloed linked a pull request Sep 15, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working as expected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants