- Google font integration: 72 lines of code for this in theme.liquid. Recommend putting into snippet called google-fonts.liquid. Tested all Google fonts, all are working!
- Google font integration: Font shrinks significantly when the "Capitalize" field is checked. Before: https://screenshot.click/08-54-r62l7-tbi92.png After: https://screenshot.click/08-54-vb7l2-gbp1z.png
- Task 23 "Alternate product page template" - Not seeing this in settings or code.
- Empty div if collection sorting and filtering not enabled. Needs condition if both are turned off. https://screenshot.click/08-11-w252p-g7e7q.png
- Alternate product card - tablet view: Product title/price cut off. https://screenshot.click/08-57-dva0o-n83wy.png
- Alternative product card - tablet view: Overlay should be turned off for tablet similar to mobile view, and for both, use title/price underneath the product. https://screenshot.click/08-59-mby6t-15845.png
- If "Enable sidebar" is checked in "Collection page" settings and "Enable sorting" is unchecked, the "Sort by" section still appears. This snippet requires a condition for this. https://screenshot.click/08-05-ms1da-81pzb.png
- "Sort by" select box needs some padding. https://screenshot.click/08-00-cc01s-33x1b.png
- Tablet view: When using Header Format "Minimal: short and sticky", there is not enough space for all elements; logo/store name overlapping navigation. https://screenshot.click/08-40-h92z3-zui3a.png - https://screenshot.click/08-49-5gqeo-3r3vv.png - https://screenshot.click/08-39-r2fah-vemcq.png
- Header social icons: Social icons are added to top left above logo; throws off vertical alignment of the cart/search area and makes overall header look unbalanced. Could put icons above cart/search area when logo is left-aligned. Looks fine when logo is centered. Would look even better if social icons and cart/search area were vertically centered to line up with logo. https://screenshot.click/08-52-okuva-bs73a.png - https://screenshot.click/08-57-2gg17-2cxq0.png
- Live search: Only noticeable difference from current demo is that search box appears in top left now instead of over the cart/search icons. There is now a disconnect between the search icon and the search box due to the relocation of the search bar. Also, there is padding missing around the search icon in search box. Also, search results are without a content wrapper (.content-wrapper) causing the results to be 100% width of screen with no margins. https://screenshot.click/08-46-3quys-a8fql.png - https://screenshot.click/08-48-e2k9y-6cz8c.png
- Live search - mobile view: Needs additional styling. https://screenshot.click/08-49-edkdz-jmog5.png
- Alternate footer: Possible alignment issue - https://screenshot.click/08-01-cbyv9-alqeg.png - Left column (.newsletter) is 50% width; right column (.footer-menu) is 33%; content not centered if secondary footer menu not enabled. When it is enabled, widths are 50+33+33. Recommend adding conditional widths when certain footer items are turned on/off. Also, when newsletter is not enabled, the center border disappears: https://screenshot.click/08-15-909oa-s1h7m.png
- Alternate footer - tablet view: Secondary footer menu too small https://screenshot.click/08-15-j0btb-456v6.png
- Sale/sold out/new badges - tablet view: Sale and new badges look off with image being smaller. Also, with sidebar enabled, On Sale badges are too big for the image. Also, when both New and On Sale badges enabled at same time, too big for image with or without sidebar. https://screenshot.click/08-19-y7etv-qahjg.png - https://screenshot.click/08-23-fg7ky-fysc7.png
- Line 37:
ul, ol { margin: 0; padding: 0 0 0 0px; display:block; list-style:none; }
- Usepadding: 0;
instead ofpadding: 0 0 0 0px;