-
Notifications
You must be signed in to change notification settings - Fork 12
Marketing changes #448
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
Marketing changes #448
Conversation
…, Husky A300 User Manual (Added Unboxing Video Guide)
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
|
Thanks for the updates. My only note is that the Husky A300 AMP looks really small on the home page when lined up beside the other robots, so maybe we can at least trim off the borders as much as possible so that it fills more of the available space. It also looks squished horizontally, which we need to fix for sure. |
|
Thanks for reviewing! I've been experimenting with the Husky AMP image locally but it seems the most we can do is trim off the borders for it to be slightly bigger. The squishing seems to be responsively happening due to the image's width to height ratio and I haven't figured out how to edit the image to accommodate for that... I will continue trying to find a solution to the squishing but have updated the image with its border trimmed for now. |
|
@ltran-cpr the feature.tsx file is enforcing both height and width for all of the images. If you would like the image aspect ratio to not be affected then only set either height or width, whichever you want to control for and remove the other line. There are other ways to fix it but this way will allow more flexibility for when images are updated in the future. |
… Jackal. + Adjusted home page header margin for better spacing
|
@hilary-luo I see, thank you for letting me know! I adjusted the sizing in the features.tsx file for the AMP, alongside a few smaller edits for Jackal, Dingo and the home header spacing. Let me know if there are any further issues before this can be merged! |
Home page
Husky A300 User Manual