Skip to content
This repository has been archived by the owner on Oct 28, 2024. It is now read-only.

Custom icons for different types of notifications. #178

Merged
merged 6 commits into from
Dec 21, 2022
Merged

Custom icons for different types of notifications. #178

merged 6 commits into from
Dec 21, 2022

Conversation

florian-obernberger
Copy link

This PR adds unique icons to the different notification types. In particular the following types got custom icons:

  • Favorites
  • Reblogs / Boosts
  • Mentions / Comments
  • Follows
  • Poll

Below you can see examples of all of them except the polls.

Favorite icon in status bar Mention icon in status bar Reblog icon in status bar Follow icon in status bar

Favorite icon in status bar

Mention icon in status bar

Reblog icon in status bar

Follow icon in status bar

Favorite icon in notification pll down Mention icon in notification pll down Reblog icon in notification pll down Follow icon in notification pll down

Favorite icon in notification pll down

Mention icon in notification pll down

Reblog icon in notification pll down

Follow icon in notification pll down

@sk22 sk22 merged commit ce2869d into sk22:main Dec 21, 2022
sk22 added a commit that referenced this pull request Dec 21, 2022
* Add filled variant of the person_add icon
* Add mention icon
* Add custom icons for different notification types

Co-authored-by: sk22 <[email protected]>
@sk22
Copy link
Owner

sk22 commented Dec 21, 2022

many thanks!!

@florian-obernberger
Copy link
Author

you're very welcome! i hope the implementation was okay, even tho I'm not really a java dev!

I was also thinking about adding an option in the settings to enable / disable it, but my knowledge of the codebase and java is not (yet?) good enough to do that.

@sk22
Copy link
Owner

sk22 commented Dec 21, 2022

oh me neither… i mean i did do a lot of java back in school, but never really lots of android. this project just kinda forced me back into it - like, it just started by adding back the federated timeline, which was simple enough 😅
tbh, i don't think a setting option for this will be really necessary, so all good!

@sk22
Copy link
Owner

sk22 commented Dec 21, 2022

also, thanks for discovering that fluent icons do actually include an @ icon - for some reason, upstream uses fluent icons for virtually anything except the @ symbol

@florian-obernberger
Copy link
Author

florian-obernberger commented Dec 21, 2022

oh me neither… i mean i did ...

That's nice tho!! I am more of a Flutter dev, but I'll have to get used to Android native anyways because some things just can't be done in Flutter (Notifications for example).

also, thanks for discovering that fluent icons do actually include an @ icon - for some reason, upstream uses fluent icons for virtually anything except the @ symbol

Yep haha. Since I already added the icon to the drawables it should be an easy switch now. There's also already a PR (#476) upstream, that does exactly that, you could probably also just merge that.

Ps.: You can also search through the fluent icons here just in case you didn't know already.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants