@adriiiiiix
Forester Erick
@Forester04All comments
- @Forester04
Nicely done
- @larryQuaoWhat are you most proud of, and what would you do differently next time?
I was able to learn and implement SASS. It was an interesting experience designing in SASS because it make things much easier.
What challenges did you encounter, and how did you overcome them?Understanding the use of some of the SASS terminologies but was able clarify everything.
What specific areas of your project would you like help with?For now, any help will be appreciable.
@Forester04Good job
- @karim1safan@Forester04
Great job. Just a small reminder: You forgot to include the background image pattern in the body to match the design. You can use the background-image property for that, and adjust it using background-size to ensure it scales properly. Make sure to position it correctly with background-position as well. This will help achieve the final look of the design. A quick best practice tip: it's always a good idea to include any external font links just before the link to your CSS file in the <head> section. This ensures that the fonts are loaded before your styles are applied, helping the page render correctly and avoiding layout shifts.
- @bajwacodesWhat are you most proud of, and what would you do differently next time?
Toughest challenge I faced, took my whole time and energy but worth it
@Forester04Good job on the project! I noticed that on mobile and medium-sized screens (around 500-600px), the background image repeats. To improve the design on smaller screens, you might want to set background-repeat: no-repeat;. This will prevent the background from repeating and make it look cleaner across different screen sizes. Keep up the great work!
- @zainy2401@Forester04
Kindly consider the following recommendations:
- Use relative units like rem/em instead of px for better scalability.
- In .hover-container, use appropriate grid properties instead of flexbox properties for alignment.
- Apply a mobile-first approach for responsive design.
Marked as helpful - @Shikamaru007@Forester04
Good job. You forgot to add the hover effect on the Sign Up button.
Marked as helpful - @donSebamarquezWhat are you most proud of, and what would you do differently next time?
Maybe the use of BEM methodology and meta tags, it's my first using them but i think i did a good job, if you have a tip or tell me another way of doing it or declaring things, you can do it in the comments and i will be thankful with you!!.
What challenges did you encounter, and how did you overcome them?Declaring things in BEM, i mean it's not so complicated but it's weird to declare, don't know how to explain it, but after some time i started typing and i think i get it now, i hope it's okay
What specific areas of your project would you like help with?Maybe on BEM, if i wrote in a way that's unnaceptable or something like that just tell me.
@Forester04Nice work.
- @romannst@Forester04
Good job. CSS file is well-organized with clear comments, concise class structures, and effective use of BEM naming conventions, improving readability.
- @Xiang511What are you most proud of, and what would you do differently next time?
Try to use better html structure
What challenges did you encounter, and how did you overcome them?Since I use flexbox, there is extra space above the card image and below . I don't know how to solve it.
What specific areas of your project would you like help with?Since I use flexbox, there is extra space above the card image and below . I don't know how to solve it.
@Forester04Good job.
- @h3cat3What are you most proud of, and what would you do differently next time?
My intention was to have an simple and easy to read code. I'm definitely improving but there are still some repetitive code
What challenges did you encounter, and how did you overcome them?The biggest challenge was learning and understanding the Flex Layout and the ways it affected all the items in the container.
What specific areas of your project would you like help with?I'm not sure how is acting on smaller screens.
- @ScarletPixie@Forester04
Nicely done.
- @Forester04@Forester04
Kindly review@Øystein Håberg
