Hi, what do you think about my work?

Hi, here’s the Survey Form that i made for the “Responsive Web Design Challenges”.

I would simply like to know what you think about it…

Thank for your answers !


CSS looked kinda cool.

1 Like

Yep, it is in french, it’s for some bass courses, sorry about that ^^

Thanks for your answer !

1 Like

Hey @Mindwarp!

I think your form looks good. There is nothing wrong with it being in french. The forum is very welcoming to all languages.

There are some errors in your html and css that need your attention. Run your code through the analyzers to work on those issues.

I think the design and layout looks nice.

Keep up the good work.

Happy coding!

1 Like

Thanks for your answer, there is so many thing to assimilate, not that easy. :smiley:

I will check the errors and i will fix it ! :slight_smile:

1 Like

No worries.

It is all part of the learning process.

1 Like

Hey @Mindwarp!
Your project looks clean and simple.

Thanks and Happy Coding!

1 Like

The code looks good only on a desktop. Check the mobile version and you find some problems:

  1. In responsive design you should change the text size.
  2. Try to change the BG of the form. For example, you can make it wider.
  3. If you use labels, use them for all input (it’s about ux).

But ui/ux from 2010 :upside_down_face:

1 Like

Hey @IhorKo!
Welcome to FCC’s Forum

1 Like

Hi @codely

Thanks for your answer !

Hi, @IhorKo !

You right, i didn’t checked on my phone if it was good looking.
Have you some good references for UX/UI ?

I started two weeks ago, so i dont really know about actuals design movement, just focusing for the moment on HTML/CSS :slight_smile:

Thanks for your answer !

For UI/UX you can find some ideas here Collect UI or here is helpful article medium.
In any case, a designer should be responsible for UI/UX :wink:

1 Like

Nice thanks again :wink:

Looks nice and functional. Good job! :sunglasses:

1 Like

@Glitches777 Thanks :slight_smile:

Looks nice.

  1. The first 3 labels are not connected to the inputs. Add the for attribute, or nest the inputs inside the labels.

  2. Add resize: vertical to the textarea input element so it can not be pulled wider by the user.

  3. Add cursor: pointer to the submit button.

  4. I would give the form the same amount of padding on the bottom as you have on the top. I’d maybe also make it 100% wide at small screen widths.

Good work, keep it up!

1 Like

Hi @lasjorg !

Thanks for your answer, i corrected the details you listed !
It’s better now, but why do we need to connect the labels to the inputs ?

Thanks again :slight_smile:

For accessibility and because it’s just good UX that the user can click the label to focus the input element.

1 Like

Ok i understand @lasjorg , i’ll work on it !
Thanks for the advices and link :wink: