Skip to content

Conversation

@iamgabrielma
Copy link
Contributor

@iamgabrielma iamgabrielma commented Jul 7, 2025

Part of WOOMOB-534

Description

This PR switches the images in woocommerce-ios/docs/ for optimized versions of the same image, as well as updating apicredentials-location.png, reducing their original size by ~75%

woocommerce-ios/docs/images/apicredentials-location.png
woocommerce-ios/docs/images/yosemite_general.png
woocommerce-ios/docs/images/storage.png
woocommerce-ios/docs/images/networking.png
woocommerce-ios/docs/images/yosemite_actions.png

We also remove the following unused assets:

woocommerce-ios/WooCommerce/Resources/Images.xcassets/woo-store-switcher.imageset
woocommerce-ios/WooCommerce/Resources/Images.xcassets/woo-mobile-sync.imageset
woocommerce-ios/WooCommerce/Resources/Images.xcassets/icon-scan.imageset

Testing information

  • Documentation under the main project's page, networking.md, yosemite.md, and storage.md should still display the images.

Screenshots

  • Example of main page using the new image for credentials:
Screenshot 2025-07-07 at 15 20 40

@iamgabrielma iamgabrielma added type: task An internally driven task. type: documentation A request for better documentation. category: tooling Anything that involves building & maintaining the project, including scripts, `Fastfile`, etc. labels Jul 7, 2025
@iamgabrielma iamgabrielma added this to the 22.8 milestone Jul 7, 2025
@iamgabrielma iamgabrielma marked this pull request as ready for review July 7, 2025 08:22
@wpmobilebot
Copy link
Collaborator

App Icon📲 You can test the changes from this Pull Request in WooCommerce iOS Prototype by scanning the QR code below to install the corresponding build.

App NameWooCommerce iOS Prototype
Build Numberpr15874-627b9bd
Version22.7
Bundle IDcom.automattic.alpha.woocommerce
Commit627b9bd
Installation URL34rg0i00bprn8
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

Copy link
Contributor

@staskus staskus left a comment

Choose a reason for hiding this comment

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

woocommerce-ios/docs/images/apicredentials-location.png

✅ Still appears in README

woocommerce-ios/docs/images/yosemite_general.png

✅ Still appears in YOSEMITE.md

woocommerce-ios/docs/images/storage.png

✅ Still appears in STORAGE.md

woocommerce-ios/docs/images/networking.png

✅ Still appears in NETWORKING.md

woocommerce-ios/docs/images/yosemite_actions.png

✅ Still appears in YOSEMITE.md

We also remove the following unused assets:

✅ yes, these assets weren't used

@iamgabrielma iamgabrielma merged commit b504c70 into trunk Jul 7, 2025
29 checks passed
@iamgabrielma iamgabrielma deleted the task/reduce-documentation-image-sizes branch July 7, 2025 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: tooling Anything that involves building & maintaining the project, including scripts, `Fastfile`, etc. type: documentation A request for better documentation. type: task An internally driven task.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants