add task solution

[KarolinaWiktoria]

• DEMO LINK
• TEST REPORT LINK

[Radoslaw-Czerniawski]

Looks very good, needs some tinkering to meet the requirements

[Radoslaw-Czerniawski]

Task requirements specify that you should use types attributes, even though inputs by default are type='text' mark it like so explicitly

[Radoslaw-Czerniawski]

Also you forgot about some of these attributes:

[Radoslaw-Czerniawski]

This way of wrapping with label is kinda unorthodox, I'd suggest in the future do it the "regular way" without wrapping and using 'for' and 'id' attributes for the sake of readability,

[Radoslaw-Czerniawski]

For now just add an empty line between all label titles and inputs e.g.:

<label class="input-row">
  Surname:

  <input name="surname" autocomplete="off">
</label>

Do it for all your labels with inputs

[Radoslaw-Czerniawski]

Fix the indent:

<label>
  <input
    type="radio"
    name="cats"
    value="no"
  > 

  No
</label>

[Radoslaw-Czerniawski]

Last change and we'r good :)

[Radoslaw-Czerniawski]

Input type range should have min and max attributes rather than minlength and maxlength, these are more appropriate for inputs with text, for example you want you password to be of some minimal length.

[KarolinaWiktoria]

Okay :). Thank you. I hope now everything is correct :-)

[Radoslaw-Czerniawski]

I see on the demo that you've adjusted it correctly but I don't see the changes in this pr for some reason, I think you commited these changes to wrong branch or something.

[KarolinaWiktoria]

hmmm... no Idea ;). Hopefully now you can see it in PR..

• DEMO LINK
• TEST REPORT LINK

Thank you and have a good weekend! :)

[Radoslaw-Czerniawski]

Good job, looks perfect :)

[KarolinaWiktoria]

🙏🏼finally 💪🏼😀