Nathan Akpawu
@natho19All comments
- @sfschiavettoSubmitted 4 months ago@natho19Posted 4 months ago
Hi π, you did a great job accomplishing this challenge. The code is clean and well-written. The mockup was respected...
However, if we want to go into details, I have a small suggestions for you :
- In the mobile version there is a padding on the card that prevents it from being stuck to the body
- The texts have different font-sizes in the mobile and desktop versions.
Apart from these small suggestions, the work is excellent π.
Keep up the good work π.
Marked as helpful0 - @KKajetSubmitted 10 months ago@natho19Posted 4 months ago
Hi π, you did a great job accomplishing this challenge. The code is clean and well-written. The mockup was respected...
However, if we want to go into details, I have a small suggestion for you regarding the box-shadow:
- The box-shadow of the card in the mockup is slightly different from yours.
#flex-center { -webkit-box-shadow: 0px 10px 100px -50px rgb(0, 0, 0); box-shadow: 0px 10px 100px -50px rgb(0, 0, 0); }
In the mockup, we have a box-shadow that is more towards the bottom with less blur and spread.
Apart from this small suggestion, the work is excellent π.
Keep up the good work π.
0 - @AhmadaldwairiSubmitted 4 months agoWhat are you most proud of, and what would you do differently next time?
In this projects, I tried to use only what I have learned so far using basic layout properties (width, height, padding, margin, boarder) without the use of flexbox, positioning, or grid.
What challenges did you encounter, and how did you overcome them?I encountered many challenges including: giving the elements the right width and height, especially that I have not learned the media query yet.
What specific areas of your project would you like help with?Since it is my first project, I still feel that my code is not clean and the approach I took was not the best approach. I know with flexbox, positioning, or grid, it will be easier to layout my elements, but I do need help making basic designs with basic properties so I feel more comfortable with CSS.
@natho19Posted 4 months agoThis seems correct to me. One point I noticed is that the box-shadow is missing on the card. Have a good day !
Marked as helpful0