search

Byabasaija / positioning-and-floating-elements-project1

4 stars 0 forks source link

Peer-to-Peer Code Review Session #7

Open Andrik27001 opened 4 years ago

Andrik27001 commented 4 years ago

Leave your questions below!

Honda99 commented 4 years ago

The buttons shouldn't be inside of the label you have to separate them.

apricot12 commented 4 years ago

Otherwise, good work!

wandji20 commented 4 years ago

1) You could reduce the form width to match the content size or more 2) give the form elements some margin so they can be more visible.

Honda99 commented 4 years ago

and don't use "br" element to create new line.

Byabasaija commented 4 years ago

Thanks for the feedback, i will work on the suggested changes.