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
@BadGuitarist07All comments
- @DeclanLaceySubmitted 10 months ago
- @MundiaNderiSubmitted 11 months ago
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.
@BadGuitarist07Posted 11 months agoHi, 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 helpful1 - @xStephxSubmitted 11 months ago@BadGuitarist07Posted 11 months ago
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 - @xStephxSubmitted 11 months ago
- @TimmyOGHSubmitted 11 months ago
Any encouraging feedback is appreciated.
@BadGuitarist07Posted 11 months agoBien hecho, buen diseño, responsive. Quizá agregar alguna transición al abrir cada pregunta para que no se vea tan tosco.
0 - @AzizJatoiSubmitted 11 months ago
I would appreciate it if anyone could review my code. Please tell me even tiny mistakes
@BadGuitarist07Posted 11 months agoEstá bien hecho, el unico inconveniente que veo, es que la img del fondo no estpa ocupando el 100% del width de la pantalla.
0 - @tanyas27Submitted 11 months ago
Feedback welcome over areas of improvement and best practices.
@BadGuitarist07Posted 11 months agoTen cuidado con la img de fondo, ya que no está ocupando el 100% de la pantalla, de resto, muy buen trabajo.
1 - @Alex-coding-3420Submitted 11 months ago
Thank you for looking at my solution for this project. Please feel free to leave any feedback regarding my solution!
@BadGuitarist07Posted 11 months agoEstá bien, solo que no es responsive. Solo sería eso, de resto, buen trabajo.
0 - @sandbrockSubmitted 11 months ago
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.
@BadGuitarist07Posted 11 months agoSi 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 - @rodjokerSubmitted 11 months ago
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?
@BadGuitarist07Posted 11 months agoSi 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 - @iDAMZYSubmitted 11 months ago
I had a little problem with the background image and I will fix that soon.
@BadGuitarist07Posted 11 months agoTen cuidado con las imagenes, ya que el fondo no se muestra en toda la pantalla. De resto, buen trabajo.
0