Hello @ArdaBozan! Good job on this challenge! Here's my few suggestions:
- Don't separate your HTML class names with "|". Separate the class names with a space, e.g.
<div class="class1 class2">. - I'd use only one paragraph for your
.priceand.opacitytext. Wrap opacity text intospanelement and style it. This will also get rid of.price-blockdiv. - Your
<button>element should be<a>anchor. Links take the user to a new location, such as a new web page or new section of the current page, buttons trigger some action, such as showing content on the page that was previously hidden, playing a video, or submitting a form. To me "Sign up" will take user to new page.
Have fun!
Marked as helpful
1
Arda Bozan• 160
@ArdaBozan
Posted
@tediko
Thanks for the advice! The reason I separate classes in that way is because I heard it from someone experienced who said it didn't cause any issues, and the reason for separating them is to distinguish between a class I've previously created and a new one I'm going to create. And yes, yes :D I didn't establish the link because it's just a design.
0
