Skip to content

feat: add an avatar size control for author bios#2647

Open
laurelfulford wants to merge 2 commits intotrunkfrom
fix/increase-archive-avatar
Open

feat: add an avatar size control for author bios#2647
laurelfulford wants to merge 2 commits intotrunkfrom
fix/increase-archive-avatar

Conversation

@laurelfulford
Copy link
Contributor

@laurelfulford laurelfulford commented Mar 9, 2026

All Submissions:

Changes proposed in this Pull Request:

This is a belated avatar size control for the theme to fix a wall I keep running into building some designs. It changes the avatar size in both the single post footer and on the author archive pages, in addition to CSS also controlling the image size. It sets the default to 120px, but should still display as 80px on single posts.

How to test the changes in this Pull Request:

  1. Apply the PR.
  2. Make sure you have at least one author with Biographical Info populated in their profile, so their bio shows on single posts.
  3. View the author profile on a single post and confirm the avatar still displays as 80px (even though it's 120px under the hood).
  4. Switch your site to use Newspack Sacha and repeat step 3.
  5. View the author profile on an archive page and confirm the avatar still shows as 120px.
  6. Navigate to the Customizer > Author Bio Settings and confirm you now have a control to change the author avatar size with a default of 120px.
CleanShot 2026-03-09 at 13 34 22
  1. Change the size of the avatar and save changes.
  2. Repeat steps 3, 4, 5 and confirm the avatars have picked up your new image size, but they still display the same size.
  3. Try changing the avatar size one more time to a value under 32 or over 512; save.
  4. Refresh the Customizer; confirm the saved value is either 32 (if you went under) or 512 (if you went over).

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

@laurelfulford laurelfulford requested a review from a team as a code owner March 9, 2026 17:33
@laurelfulford laurelfulford requested review from Copilot and removed request for a team March 9, 2026 17:33
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new Customizer control to configure the requested author avatar image size, and applies it consistently across author bio templates and author archives to support more flexible design needs in Newspack themes.

Changes:

  • Introduces a new author_avatar_size theme mod (default 120px) in the Customizer under Author Bio Settings.
  • Updates avatar retrieval in the Newspack and Newspack Sacha author bio templates to use the new setting.
  • Updates author archive header avatars to use the new setting.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
newspack-theme/template-parts/post/author-bio.php Uses author_avatar_size when requesting author avatars in single post author bio output.
newspack-theme/inc/customizer.php Adds the Customizer setting/control for author_avatar_size (default 120).
newspack-theme/archive.php Uses author_avatar_size when requesting author avatars on author archive headers.
newspack-sacha/template-parts/post/author-bio.php Mirrors the author bio avatar-size behavior for the Sacha child theme.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@laurelfulford laurelfulford added the [Status] Needs Review The issue or pull request needs to be reviewed label Mar 9, 2026
@laurelfulford laurelfulford requested a review from a team March 9, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Status] Needs Review The issue or pull request needs to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants