Alin Costea• 190
@CosteaAlin93
Posted
Hello Jafar ! Nice work :) A few remarks:
- start designing with a 'Mobile-first' perspective, this way it's easier to adjust the elements for bigger screens once mobile view is done
- on the Mobile design, you should reduce the width of the cards, so you are 1:1 with the design and also because once I make my window really small, the p text is not properly visible
- the h1 font family should be 'Big Shoulders Display'
- and on the Desktop view, same thing, you should set the width of the <p> element so ~150px, so the text wraps nicely within the container, else it gets 'under' it's neighbour element
Keep up the good work, let me know if you have and doubts about my answer :)
Marked as helpful
0
jafar-mirzapoor• 170
@jeff-dev0
Posted
hello, @CosteaAlin93 thank you for your time and review I hope someone checks my projects and notices me with my problems .next time I will try to have a clean and improved code and design. Appreciate it 🙏
0
Alin Costea• 190
@CosteaAlin93
Posted
@jeff-dev0 followed you now, will keep taking a look at your already posted projects and review when possible :)
0
jafar-mirzapoor• 170
@jeff-dev0
Posted
@CosteaAlin93 thanks it means a lot🙏
0
