toxic icon indicating copy to clipboard operation
toxic copied to clipboard

Implement Tox Client Standard concerning the avatars

Open ovalseven8 opened this issue 9 years ago • 1 comments

2.3.5 Clients must display avatars at a 1:1 aspect ratio.

Not relevant for Toxic at the moment because the display of avatars is not supported yet.

2.3.6 A client may only transmit avatars with an aspect ratio of 1:1. If the user has chosen a non-square avatar, the client must transmit the square portion of the avatar displayed to the user. In the case that the user's client doesn't display avatars, the client must pad the avatar evenly on two sides with transparent rectangles, in order to create a square.

That's relevant for Toxic. What's possible as well is that users are only able to set a square avatar. So, you wouldn't need to pad avatars.

ovalseven8 avatar Nov 13 '15 19:11 ovalseven8

Well, let's pick this back up and check compatibility since toxic supports avatars now.

Unfortunately, the feature does not work for me - I tried to set tox.avatar_path="/Path/to/gendry.png" with the following PNG, which is far from the size limit: gendry But get

Failed to set avatar. Avatars must be in PNG format and may not exceed 65536 bytes.

Okay, figured it out while typing - I forgot a / at the beginning of the path. Would be nice if the error message would be different for an invalid file path.

xeruf avatar Jan 25 '22 23:01 xeruf

This has been implemented

JFreegman avatar Jan 25 '24 18:01 JFreegman