KTS915 / Test-ClassicPress-Directory

GNU General Public License v3.0
0 stars 0 forks source link

Remove Gravatars #2

Closed viktorix closed 2 years ago

viktorix commented 2 years ago

For privacy, we should consider removing Gravatars. If an avatar is needed, we should consider adding local avatars: https://directory.classicpress.net/plugins/avatars

KTS915 commented 2 years ago

I'm happy to do that but, if we're going to use avatars at all, I have my own avatars plugin that I much prefer and use on many of my sites. It uses an earlier filter and so is much more efficient, as well as providing a built-in SVG letter fallback.

The question really is whether we want to have avatars at all.

viktorix commented 2 years ago

It's probably more of an aesthetic choice. Since it represents a developer/brand, it could be their logo, and it could be shown on their public page as well as an icon on plugin/theme pages to make it looks visually pleasing.

Your avatar plugin is good, too.

xxsimoxx commented 2 years ago

+1 for your avatar plugin.

KTS915 commented 2 years ago

Addressed by above commit and associated mods.

viktorix commented 2 years ago

Works.