Reader: Fix cropped recommended sites

[kean]

Before:

After:

Note: I added 16pt vertical spacing (not visible on the screenshots)

Simulator.Screen.Recording.-.iPhone.16.Pro.-.2024-11-13.at.17.00.04.mp4

It also uses a different design for the "Subscribe" buttons to make sure site titles fit.

To test:

Regression Notes

1. Potential unintended areas of impact

2. What I did to test those areas of impact (or what existing automated tests I relied on)

3. What automated tests I added (or what prevented me from doing so)

PR submission checklist:

• I have completed the Regression Notes.
• I have considered adding unit tests for my changes.
• I have considered adding accessibility improvements for my changes.
• I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Testing checklist:

• WordPress.com sites and self-hosted Jetpack sites.
• Portrait and landscape orientations.
• Light and dark modes.
• Fonts: Larger, smaller and bold text.
• High contrast.
• VoiceOver.
• Languages with large words or with letters/accents not frequently used in English.
• Right-to-left languages. (Even if translation isn’t complete, formatting should still respect the right-to-left layout)
• iPhone and iPad.
• Multi-tasking: Split view and Slide over. (iPad)

[dangermattic]

	2 Warnings
⚠️	This PR is larger than 500 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.
⚠️	This PR is assigned to the milestone 25.5. The due date for this milestone has already passed. Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by 🚫 Danger

[wpmobilebot]

📲 You can test the changes from this Pull Request in WordPress Alpha by scanning the QR code below to install the corresponding build.

	App Name	WordPress Alpha
	Configuration	Release-Alpha
	Build Number	pr23802-9acbf09
	Version	25.4.2
	Bundle ID	org.wordpress.alpha
	Commit	9acbf09
	App Center Build	WPiOS - One-Offs #11042

Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

[wpmobilebot]

📲 You can test the changes from this Pull Request in Jetpack Alpha by scanning the QR code below to install the corresponding build.

	App Name	Jetpack Alpha
	Configuration	Release-Alpha
	Build Number	pr23802-9acbf09
	Version	25.4.2
	Bundle ID	com.jetpack.alpha
	Commit	9acbf09
	App Center Build	jetpack-installable-builds #10082

Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

[crazytonyli]

Nitpick: may as well rename the label size: -> pixelSize: 😄

[kean]

It's actually size in points 🤦

This is how it's used:

  static func optimizedBlavatarURL(from path: String, imageSize: CGSize) -> URL? {
        let size = imageSize.scaled(by: UITraitCollection.current.displayScale)
        let query = String(format: "d=404&s=%d", Int(max(size.width, size.height)))
        return parseURL(path: path, query: query)
    }

I'm going to revert it.

Yeah, it's a low-level method. The app uses SiteIconViewModel.Size enum in most cases. It also ensures that you don't create too many different variants of the the same icon but off by a few points.