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

Fastfetch: new app (draft) #2673

Merged
merged 1 commit into from
Nov 2, 2024
Merged

Fastfetch: new app (draft) #2673

merged 1 commit into from
Nov 2, 2024

Conversation

theofficialgman
Copy link
Collaborator

@theofficialgman theofficialgman commented Nov 2, 2024

@Botspot any opinion on an icon to use for this? fastfetch doesn't seem to have one (I just kept the neofetch one for now).

I'll add an auto-updater assuming you have no objections.

Test run https://github.com/Botspot/pi-apps/actions/runs/11645397719

@Botspot
Copy link
Owner

Botspot commented Nov 2, 2024

I think keeping the same icon could inform users that it is to be considered similar to Neofetch, so that is not a bad thing.
Now if you do want a bit of icon distinction, you could tweak the colors a bit, or italicize the penguin to signify speed maybe. Those are just some initial thoughts of mine.

Merge when ready.

@theofficialgman theofficialgman marked this pull request as ready for review November 2, 2024 20:43
@theofficialgman
Copy link
Collaborator Author

I think keeping the same icon could inform users that it is to be considered similar to Neofetch, so that is not a bad thing. Now if you do want a bit of icon distinction, you could tweak the colors a bit, or italicize the penguin to signify speed maybe. Those are just some initial thoughts of mine.

Ok. For now I will just leave it. Maybe in the future they will have an icon for branding of their own and we can switch.

@theofficialgman theofficialgman merged commit 12c225a into master Nov 2, 2024
3 checks passed
@theofficialgman theofficialgman deleted the fastfetch branch November 2, 2024 20:47
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