@xStephxSubmitted 3 months ago
Tarachand Prakash Khorwal
@tarachand-kAll comments
- @tarachand-kPosted 3 months ago
- Font color is not correct. (provided inside style-guide.md perhaps)
- Spacing is not correct.
1 - @fivetailsdevelopmentSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
The JS button added as an enhancement to change the grid layout was a pilot for a personal project I want to build in the future, and really happy with how it turned out.
What challenges did you encounter, and how did you overcome them?Responsive font sizing is still a bit fun. Still yet to master the art of getting font scalability on point.
@tarachand-kPosted 3 months agochange the text color which are inside ul and ol lists to same which you have used for paragraphs
0 - @abdelkarim-el-manssouriSubmitted 3 months ago@tarachand-kPosted 3 months ago
Looks good. And Nice tagline at the bottom 😂
Marked as helpful0 - @DubidimiSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
ok
What challenges did you encounter, and how did you overcome them?ok
What specific areas of your project would you like help with?ok
@tarachand-kPosted 3 months agoYou should use flex layout to center the card.
1 - @danHD2Submitted 3 months agoWhat are you most proud of, and what would you do differently next time?
This seemed pretty easy. I wrote the styles in the same index file.
What challenges did you encounter, and how did you overcome them?Centering the component vertically was a small challenge, I used a top padding.
What specific areas of your project would you like help with?Just general feedback
@tarachand-kPosted 3 months agoYou could use flex layout to center the card. You should refer figma design file for spacing and typography
0