This was a really fun project to build. Took me about a week which is about how long I thought it would take me. The project often made me think outside the box especially when it came to the CSS and doing some things I had never done before.
Ricardo Vargas
@BadGuitarist07
All comments
- Ricardo Vargas• 80
@BadGuitarist07
Posted
WOW! Awesome work.
1 - Serah Nderi• 430
@MundiaNderi
Submitted
I am learning React from SheCodes and implementing Frontend Mentor designs. This was a simple and easy design that I would recommend for beginners. I would appreciate any feedback.
Ricardo Vargas• 80@BadGuitarist07
Posted
Hi, Serah
Great Job!
I would just recommend you to center The Card well to the screen, you could do it with this code in your general css file of the website
body { min-heigth: 100vh; display: flex; justify-content: center; align-items: center; }
Marked as helpful
1 - Steph• 500
@xStephx
Submitted
Ricardo Vargas• 80@BadGuitarist07
Posted
Excelente trabajo, solamente le pondría un :hover al boton y cursor: pointer al botón del "Sign Up" para darle un poco de dinamismo.
Resumiendo, quedó practicamente identico.
1 - Steph• 500
@xStephx
Submitted
Ricardo Vargas• 80@BadGuitarist07
Posted
Looks great, well done!
0 - timodn• 240
@TimmyOGH
Submitted
Any encouraging feedback is appreciated.
Ricardo Vargas• 80@BadGuitarist07
Posted
Bien hecho, buen diseño, responsive. Quizá agregar alguna transición al abrir cada pregunta para que no se vea tan tosco.
0 - Aziz• 130
@AzizJatoi
Submitted
I would appreciate it if anyone could review my code. Please tell me even tiny mistakes
Ricardo Vargas• 80@BadGuitarist07
Posted
Está bien hecho, el unico inconveniente que veo, es que la img del fondo no estpa ocupando el 100% del width de la pantalla.
0 - Tanya Singh• 10
@tanyas27
Submitted
Feedback welcome over areas of improvement and best practices.
Ricardo Vargas• 80@BadGuitarist07
Posted
Ten cuidado con la img de fondo, ya que no está ocupando el 100% de la pantalla, de resto, muy buen trabajo.
1 - Alex-coding-3420• 40
@Alex-coding-3420
Submitted
Thank you for looking at my solution for this project. Please feel free to leave any feedback regarding my solution!
Ricardo Vargas• 80@BadGuitarist07
Posted
Está bien, solo que no es responsive. Solo sería eso, de resto, buen trabajo.
0 - PBrandon Rock• 20
@sandbrock
Submitted
This challenge was difficult for me because of the requirement to hide and show elements dynamically. I was unsure whether to use an ES6 class or a JavaScript object. I went with the JavaScript object to be safe.
Ricardo Vargas• 80@BadGuitarist07
Posted
Si quieres usar algún Framework, te recomendaría Bootstrap. Se hace un poco más sencillo el hacer este tipo de ejercicios.
Te dejo el enlance para que le eches una lectura rápida, es del componente 'accordion', el cual es usado para este tipo de ejemplos.
Muchos éxitos
0 creo que lo mas complicado es tener el css adecuado para que JavaScript active y desactive la clase, aun siento que me quedo muy ancha la caja de las preguntas
¿esto hoy en día se hace con alguna librería de css o en la practica las paginas siguen usando este método?
Ricardo Vargas• 80@BadGuitarist07
Posted
Si quieres usar algún Framework, te recomendaría Bootstrap. Se hace un poco más sencillo el hacer este tipo de ejercicios.
Te dejo el enlance para que le eches una lectura rápida, es del componente 'accordion', el cual es usado para este tipo de ejemplos.
Muchos éxitos
0- Isaac Opeyemi• 10
@iDAMZY
Submitted
I had a little problem with the background image and I will fix that soon.
Ricardo Vargas• 80@BadGuitarist07
Posted
Ten cuidado con las imagenes, ya que el fondo no se muestra en toda la pantalla. De resto, buen trabajo.
0