search

UAllen95 / prj-rev-bwfs-dasmoto

0 stars 0 forks source link

Overall #3

Open Axkaban opened 6 years ago

Axkaban commented 6 years ago

Well achieved success! Good thinking on separating index.css, and try to start adding some comments, think on where do you think it will help you the most for example, maybe here: https://github.com/UAllen95/prj-rev-bwfs-dasmoto/blob/master/arts_%26_crafts/resources/css/index.css#L17-L22 and here: https://github.com/UAllen95/prj-rev-bwfs-dasmoto/blob/master/arts_%26_crafts/resources/css/index.css#L1-L3

you might need to explain to somebody the difference between those two, in the sence of why didn't just do all bold.

As extra work, separate each product by wrapping it into a div, you might have to adjust your css to keep it looking the same. Once you have done that, explore the :hover pseudo-class and change the color of the spans every time you hover over them.

UAllen95 commented 6 years ago

Awesome tips again. Will do. :)

UAllen95 commented 6 years ago

I still had paragraphs inside the html document; so being specific about everything else excluding the "p" elements helped me exclude that. Of course, there was the option to bold everything with "*" or "body" and then p { font-weight: normal; }.