Hi, I just finished my first project! I am very new to coding so any feedback is welcome Thank you
Hi @hetauda
I noticed two things with this code.
<label id="email-label" for="email">Email<input id="email" type="email" placeholder="turnicola@hotmail.com..." required></ input></lable>
input
elements do have a closing tag- there is a typo in the closing labEL tag.
There is a mismatch between the for
and id
attribute values.
<label for="leght">
<input type="radio" value="test1" name="lenght" id="lenght" class="inline">
Consider using san serif fonts, as they are considered more web friendly and modern.
Adding single spaces may help make the text easier to read.
c
Hi there!
It’s looking good. Keep it up the good work. Happy Coding.
Thank you so much for the feedback! I’ll correct the mistakes and thanks for the tips
Thank you, I amended all the things that you suggested and It looks better. Font tip was great too, I didn’t know about it. Thank
Looking good. Keep up the good work. I’ve also made a project like this.
Looks like you’ve created only one label for your three inputs by the radio group “How long do you plan to travel?”
<label for="lenght">
<input type="radio" value="test1" name="lenght" id="lenght" class="inline">Weekend (3 days)
<input type="radio" value="test2" name="lenght" class="inline">Short trip (7 days)
<input type="radio" value="test3" name="lenght" class="inline">Extended trip (8+ days)
</label>
To make them clickable, you should create a label per radio button as well as link it to its matching input id. Watch this one:
<label for="3-days">
<input type="radio" id="3-days" class="inline" name="length" value="3-days">
</label>
What do you think?