five-for-the-future icon indicating copy to clipboard operation
five-for-the-future copied to clipboard

Change rendered avatar size

Open pkevan opened this issue 1 year ago • 2 comments

Reduces the standard fetched avatar size, dependant on the format.

Fixes #317.

Note: couldn't successfully build this to test locally.

pkevan avatar Sep 13 '24 10:09 pkevan

Good catch on the image size 🙂

The sizes are reversed here — the avatar should be 30px when truncated, and 70px when not.

Screenshot 2024-09-13 at 12 18 41 PM

Note: couldn't successfully build this to test locally.

I never updated the instructions, so they're probably not correct 😅

ryelle avatar Sep 13 '24 16:09 ryelle

I just updated the directions in the readme, the biggest difference from other projects was that I'm trying out vanilla/modern npm, instead of the legacy yarn we've been using.

The new setup uses wp-env, but if you had something working with a different local env, you should be able to run these to get the new theme built and usable:

npm run setup:tools
npm run build:theme

ryelle avatar Sep 13 '24 20:09 ryelle

The sizes are reversed here — the avatar should be 30px when truncated, and 70px when not.

OK - makes sense, got confused for a minute!

pkevan avatar Sep 26 '24 14:09 pkevan