@MarlonCJSubmitted 8 months ago
P
mohamed-fathy3010
@mohamed-fathy3010All comments
- @mohamed-fathy3010Posted 19 days ago
Good job using BEM.
One thing i noticed in the code is that you don't need to use media query here. All the designs had the same width just a max-width will be enough with some more padding for the card and that's it.
other than that, nice work.
0 - @alberto-rjSubmitted 8 months agoWhat are you most proud of, and what would you do differently next time?
.
What challenges did you encounter, and how did you overcome them?.
What specific areas of your project would you like help with?.
@mohamed-fathy3010Posted 20 days agoGood job using BEM and nice animations too.
The only thing I noticed is at the first load of the page the elements show up first then disappear then the animation starts and make them appear again creating a flickering effect for a second (using google chrome).
Marked as helpful1 - @SamueleVillaSubmitted 4 months ago@mohamed-fathy3010Posted 25 days ago
Nice Job. I can see you used BEM too.
You can download Figma and use the units given in the design (like font-size - width - ...etc) to achieve a near perfect match.
0