DiscordStyles / RadialStatus

Changes the status icons to wrap around the avatar.
MIT License
111 stars 154 forks source link

Mobile Icon #6

Closed DracoClaw closed 2 years ago

DracoClaw commented 3 years ago

So it's been some time since we discussed this. I did some examples, and I think that putting the Mobile Icon back to were it is in vanilla would look the best.

image

Gibbu commented 3 years ago

The different colour is the only move here. I'm going to add 2 new variables.

--rs-phone-visible will be if the user wants to display the phone icon. Options are block for visible and none for hidden. --rs-phone-colour will be the colour of the ring and the phone icon. This will defaulted to --rs-online-color.

DracoClaw commented 3 years ago

Any chance you could also add the other option? Just for some customization

Gibbu commented 3 years ago

I cannot. The way Discord has structured their avatars and status is really really limiting. I was somewhat able to get it to work but then any time that user typed, updated their status or you clicked on their name it would make the ring go over the phone and removed the substitute background.

I also had to factor in the compatibility for main themes. If I used that method that sort of worked, themes would have to add support for RadialStatus.

DracoClaw commented 3 years ago

Couldn't you set a new path that is the ring and phone together? One shape not two.

Gibbu commented 3 years ago

No I cannot. As I said before. The way Discord structured their avatars and statuses is very limiting. The only thing I can with work is a rect element. I'm more than happy to merge a good way if you can figure one out.

Didn't meant to close the issue.