@Blazing-Mike
@paulauskas-linas
Submitted
This is my first Frontend Mentor challenge. I have tried my best to write the code as DRY as possible. I would like some feedback if I could make it more optimised. This is my first time going with mobile-first workflow and I fallen in love with this approach. Any tips and suggestions on this are very welcome!
@Blazing-Mike
Posted
This looks great! It is mobile responsive and the validation works well. If you want improve it, you can work on the password validation to accepts on >= 6 characters ( and combination of letters, numbers and symbols). The CSS looks great as well !
@Mimieveshofuo
Submitted
Feedbacks would be appreciated. Thanks.
@Blazing-Mike
Posted
Great job you did here. Everything looks perfect, the only suggestion i have is the password input; it accepts an input of even two characters, in the password validation you can add that it returns an error for password <= 6 characters. If you want to go further you can add that it contains a combination of letters or symbols and numbers. Well-done Elizabeth
Marked as helpful
@Blazing-Mike
Submitted
I TRIED MY BEST ON THIS ONE. YOUR FEEDBACKS ARE APPRECIATED
@Blazing-Mike
Posted
Hmmn it's looks a bit distorted and I need to refactor the code
@fame0100
Submitted
Suggestions and comments are welcome
@Blazing-Mike
Posted
Great attempt 🚀🚀. You should check if your body by any means has margin because there are little spaces around the page..... Did you use different images for desktop and mobile? You can use the picture element or use media queries to display different image of viewports
@EsmithAS
Submitted
Hey coders, si usted tiene algún comentario sobre mi solución, por favor deje un comentario. Muchas gracias!! 💻😎
@Blazing-Mike
Posted
grande tentative que vous avez eu ... Pixel parfait et pour détailler. continuons à nous améliorer et à construire
@mraafat2021
Submitted
Please review the word and let me know your comments, it will help me improve as I am still a student. Thanks in advance.
@Blazing-Mike
Posted
On your mobile view. The image section before the footer should have four images displayed horizontally. You can use display grid or flex for that section and also use media queries to tweak any element you want to on different viewports
LAYOUTS ( CSS GRID AND FLEXBOX) MEDIA QUERIES
@mraafat2021
Submitted
Please review the word and let me know your comments, it will help me improve as I am still a student. Thanks in advance.
@Blazing-Mike
Posted
On your mobile view. The image section before the footer should have four images displayed horizontally. You can use display grid or flex for that section and also use media queries to tweak any element you want to on different viewports
LAYOUTS ( CSS GRID AND FLEXBOX) MEDIA QUERIES
Marked as helpful
@Blazing-Mike
Submitted
feedbacks are appreciated
@Blazing-Mike
Posted
Noted. I was just focused on the countdown, I got tired of the UI. I would make changes right away
@Blazing-Mike
Submitted
I TRIED TO GET THE LAYOUT BUT IT WAS TRICKY, FEEDBACKS ARE APPRECITED
@Blazing-Mike
Posted
Thanks. I would refactor it
@DavidPorras043
Submitted
I appreciate your comments.
@Blazing-Mike
Posted
Everything works just fine 🌝. You could just align the container vertical to center on desktop mode.
With . container { Display grid; Justify Content: center; Align items : center; Min-height:100vh }
@angelicambs9
Submitted
Qualquer feedback é bem vindo :)
@Blazing-Mike
Posted
Great attempt you made. But the desktop site seem a bit distorted. You can check the height of the testimonial boxes and the spacing between the top container and bottom container
Marked as helpful
@rporfirio88
Submitted
I like feedbacks, in this way i can improve my front-end abilities! feel free to give me good constructive feedback.
@Blazing-Mike
Posted
Ok great attempt you had... Try aligning your card like give it some space top and bottom.. you can use margin tip and bottom or use
Display: grid; Align-items: center; Min-height: 100vh; Justify-content: center;
To align it vertically and horizontally
Marked as helpful
