Interactive card details form using React & Sass

Community feedback

• 

  David•7,960

  @DavidMorgade

  Posted almost 3 years ago

  Hello Vietano, congrats on finishing the challenge!

  I will try to answer your questions and give you my personal opinion on your code:

  • I think yes, 4 nested CSS selectors are hard to read, you should stick to maximum 3 to make your code more readable and easy to change / understand, try maybe using BEM to avoid excesive nesting or just call the class in another block on your SCSS.

  • Well I can understand it, but it can be more easy to read if you change a bit your habits of your Javascript style, but this can be a hard path and it will just come with time, for example:

  Instead of for of loop (wich yes, they run faster but are way less readable), use array methods like map(), forEach() etc... instead of using if else checks try using ternary operators or logical && || operators, this are modern features that can reduce your code a lot and make it more readable. Dunno how exactly Vite works, since I have only used Create react app and NextJS but you should divide your project in a lot of components, instead of having everything on one file.

  • As I said before I think using Array methods here could have done the job easily for you.

  Hope my feedback helps you, if you have any questions don't hesitate to ask, you did a great job, welcome to the react community of frontendmentor!

  14

  Marked as helpful