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

Instruct Simple Local Avatars to use Tachyon image resizing #744

Open
johnbillion opened this issue Sep 21, 2023 · 0 comments
Open

Instruct Simple Local Avatars to use Tachyon image resizing #744

johnbillion opened this issue Sep 21, 2023 · 0 comments

Comments

@johnbillion
Copy link
Member

If a user avatar is requested of a size that Simple Local Avatars doesn't already know about, it'll instantiate a WP_Image_Editor instance, attempt to resize the image, save it to disk, and update the user meta value with the updated URL.

All of this should be unnecessary on an Altis site with Tachyon in use. The URL for any given avatar size should be provided by Tachyon with the appropriate URL query parameters for the request size.

Simple Local Avatars currently lacks the necessary filters to achieve this. I've opened a PR to add one to Simple Local Avatars.

I've opened this issue on this repo because it's the one that has 10up/simple-local-avatars as a dependency. The change itself should probably go into Tachyon.

Steps to reproduce

  1. Visit your user profile screen
  2. Upload an avatar
  3. Save your profile
  4. Reload the page
  5. Observe that a {image}-52x52.jpg file has been created and saved by Simple Local Avatar and used as your avatar image
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

No branches or pull requests

1 participant