mofada• 310
@mofada
Submitted
What are you most proud of, and what would you do differently next time?
I use radio to implement chang time
Hamid Riaz
@hamidriaz1998
All comments
- Hamid Riaz• 150
@hamidriaz1998
Posted
Looking Good
0 - anthony• 170
@Codingtry123
Submitted
What are you most proud of, and what would you do differently next time?
how to simplify the toggle class
What challenges did you encounter, and how did you overcome them?
::before does not work on input use div container to wrap input
What specific areas of your project would you like help with?
how do I use validator to validate email
Hamid Riaz• 150@hamidriaz1998
Posted
Looking good! Regarding class toggling the
toggle()function has an optional second paramter that specifies the condition for toggling. For example,toggle("dark", 5 > 2);This will toggle the class if the condition is true.0 - TCO932• 220
@TCO932
Submitted
Hamid Riaz• 150@hamidriaz1998
Posted
Looking good!
0 @coding-astrounaut
Submitted
What are you most proud of, and what would you do differently next time?
I have finished the 21-hour free code camp front-end crash course. Next time I would like to do a challenge entirely by myself instead of a bit of help from the YouTube video.
Also, I think my design is a bit higher than the design given. However, I still think it looks good!
What challenges did you encounter, and how did you overcome them?
The quotation mark was hard to put behind the text but above the purple container.
What specific areas of your project would you like help with?
if I could write css or html more efficiently or at a better standard.
Hamid Riaz• 150@hamidriaz1998
Posted
Looking good!
0- tsuki - 青• 120
@tsukiyx
Submitted
Hamid Riaz• 150@hamidriaz1998
Posted
It looks great! There is just one problem that is you have not used font weights properly. You can refer to
style-guide.mdthat explains what font weights to use Keep Growing!Marked as helpful
0 - gk6294• 150
@gk6294
Submitted
Hamid Riaz• 150@hamidriaz1998
Posted
Congrats on solving the problem!
But here are some problems:
- The font size of both product name and price is small.
- The gap between elements is too small.
After fixing these issues your solution shall look good. Keep growing
0 - Jacob Allison• 40
@JacobAlli
Submitted
Hamid Riaz• 150@hamidriaz1998
Posted
Congrats on completing the problem
0 - jaliecac• 60
@jaliecac
Submitted
Hamid Riaz• 150@hamidriaz1998
Posted
Congrats on solving the problem 🎉 There is just one problem and that is the card is not centered on the display. To fix this just add this line in the css of the body tag:
body{ /* Other properties */ align-items: center; }0 What are you most proud of, and what would you do differently next time?
After getting valueable inputs from @Islandstone89 I have tried to follow his instructions. In this project I have used semantic HTML and Responsive CSS techniques.
What challenges did you encounter, and how did you overcome them?
To be Honest, Semantic HTML and structuring the CSS file so that there is no redundant style was a challenge. I have structured the CSS file so that the rule sets are grouped based on the elements they are applied for my own understanding.
What specific areas of your project would you like help with?
Dear Mentors, Thank you for your valuable input last time. I would seriously need your help in assessing my skill of basic HTML and CSS and guide me further. Please give me a feedback after assessing this project . Thank you.
Hamid Riaz• 150@hamidriaz1998
Posted
Congrats on solving the problem! Keep growing
1- Jacob Allison• 40
@JacobAlli
Submitted
Hamid Riaz• 150@hamidriaz1998
Posted
0
