hovski / ecommerce-pattern-library

A term-long project redesigning an ecommerce website focusing on making a pattern library.
https://stupefied-mccarthy-ee9958.netlify.com/pattern-library/#brand
0 stars 0 forks source link

Images not working #4

Open hovski opened 5 years ago

hovski commented 5 years ago

@thomasjbradley For some reason I can't get the images to show up on any parts of my website. If you could let me know what's up, that'd be great!

thomasjbradley commented 5 years ago

I only see one place where the products collection is being used, that’s on products.html. And there are two small typos that a preventing it from working:

https://github.com/hovski/ecommerce-pattern-library/blob/master/products.html#L12 site.products — a dot, not a dash

https://github.com/hovski/ecommerce-pattern-library/blob/master/products.html#L13 {% pattern cards/main-card data=product %} — lowercase “p” in product, to match the spelling on the line above.

hovski commented 5 years ago

I fixed this. It shows the images now for the products. My only thing is, within my Sections. There are sections that have images on grid format. Those images show as broken image icons not the images themselves and I’m not sure how to resolve this issue.

Sent from my iPhone

On Mar 20, 2019, at 4:01 PM, Thomas J Bradley notifications@github.com wrote:

I only see one place where the products collection is being used, that’s on products.html. And there are two small typos that a preventing it from working:

https://github.com/hovski/ecommerce-pattern-library/blob/master/products.html#L12 site.products — a dot, not a dash

https://github.com/hovski/ecommerce-pattern-library/blob/master/products.html#L13 {% pattern cards/main-card data=product %} — lowercase “p” in product, to match the spelling on the line above.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

thomasjbradley commented 5 years ago

I don’t see any code within the different tea sections—like what was written on products.html that would output the tea cards into those patterns.

hovski commented 5 years ago

Yeah, see that’s part of my design problem. Each card is just a static image of any tea type with its hoverstate. The only time teacard content should show similar to the products page would be on the section another student would be designing. My sections are just images of tea. When you click them it goes to /product-description/ page not my products page. I don’t know if I’m making sense.

Sent from my iPhone

On Mar 20, 2019, at 4:26 PM, Thomas J Bradley notifications@github.com wrote:

I don’t see any code within the different tea sections—like what was written on products.html that would output the tea cards into those patterns.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

thomasjbradley commented 5 years ago

On the products.html page (after you’ve made the above fixes) how is what’s being output at the bottom different than what you want in the different tea colour sections?

hovski commented 5 years ago

So the best way to describe my problem is this.

If I were to go to my localhost/products/betty-white/ lets say. The image shows up, the content for that card shows up. Everything shows up, but not inside my banner where it should be (which is fine for now, but it is supposed to be showing up inside one of my banners called product-description.) My product-description stuff (I've realized is the page a student would be making, but since one assignment wanted us to do all the .md things I ended up making content for my cards).

My homepage/index has nothing but 1 banner, and 1 carousel (carousel content is just an image which should be working but isn't. It used to work until I made these .md files) And my products page is just a page with sections of 8 different teas per section, but those images also don't work.

The student does the other page that goes to the item and explains what the item is. I'm going to attach photos below to explain this better. The image with the arrow is the page the student would make (and you'll see why I did the .md's/content for that page since my pages don't have any descriptions of my products on either index or products.

On Wed, Mar 20, 2019 at 6:48 PM Thomas J Bradley notifications@github.com wrote:

On the products.html page (after you’ve made the above fixes) how is what’s being output at the bottom different than what you want in the different tea colour sections?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/hovski/ecommerce-pattern-library/issues/4#issuecomment-475058758, or mute the thread https://github.com/notifications/unsubscribe-auth/AeYGliMOsRwPEHyspiNg43Kb0cr5gTS3ks5vYrrJgaJpZM4cAIJQ .

thomasjbradley commented 5 years ago

The photos don’t come through if you attach them to an email response. You have to upload them directly to GitHub.

hovski commented 5 years ago

unnamed 12312321

thomasjbradley commented 5 years ago

Yes, I completely understand all that.

What I’m trying to say is that your products page tea secions don’t work because you haven’t written any code yet! If you look currently at the products.html file and view it in the browser, what’s outputting at the bottom of that page? All the products from the MD filesand only their image.

The images within the tea sections don’t work because you haven’t used the code you wrote in the products.html within each of those tea sections to output the images.

hovski commented 5 years ago

Holy moses you're a genius. I didn't even realize this till you told me. Ooof, thank you.

On Wed, Mar 20, 2019 at 7:20 PM Thomas J Bradley notifications@github.com wrote:

Yes, I completely understand all that.

What I’m trying to say is that your products page tea secions don’t work because you haven’t written any code yet! If you look currently at the products.html file and view it in the browser, what’s outputting at the bottom of that page? All the products from the MD filesand only their image.

The images within the tea sections don’t work because you haven’t used the code you wrote in the products.html within each of those tea sections to output the images.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/hovski/ecommerce-pattern-library/issues/4#issuecomment-475065619, or mute the thread https://github.com/notifications/unsubscribe-auth/AeYGlhYIiiPA7Rw8CqtvSrUb6rgXrfaiks5vYsI-gaJpZM4cAIJQ .

hovski commented 5 years ago

So I'm clearly not getting what you've explained to me. Can I meet up with you to go over this coding issue?

On Wed, Mar 20, 2019 at 7:26 PM Annie Pandolfi ampdepictions@gmail.com wrote:

Holy moses you're a genius. I didn't even realize this till you told me. Ooof, thank you.

On Wed, Mar 20, 2019 at 7:20 PM Thomas J Bradley notifications@github.com wrote:

Yes, I completely understand all that.

What I’m trying to say is that your products page tea secions don’t work because you haven’t written any code yet! If you look currently at the products.html file and view it in the browser, what’s outputting at the bottom of that page? All the products from the MD filesand only their image.

The images within the tea sections don’t work because you haven’t used the code you wrote in the products.html within each of those tea sections to output the images.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/hovski/ecommerce-pattern-library/issues/4#issuecomment-475065619, or mute the thread https://github.com/notifications/unsubscribe-auth/AeYGlhYIiiPA7Rw8CqtvSrUb6rgXrfaiks5vYsI-gaJpZM4cAIJQ .