@Ah-EssamSubmitted over 1 year ago
Firoozeh Imany
@firoozehImanyAll comments
- @firoozehImanyPosted over 1 year ago
👋 Hi, Ahmed
🎉Congratulations, your solution looks great. 🎉
I want to tell you the things that come to my mind to improve the solution. If you apply this change to it, your solution will be better
CSS:
.header-container { padding: 32px; width: 350px; } .header-container .circle { height: 200px; width: 200px; justify-content: center; } .header-container .circle h3 { font-size: 70px; } .header-container>p { max-width: 230px; } .container { padding: 32px; width: 350px; } .container .content .con-icon { display: flex; align-items: center; } .container .content>p { color: hsla(224, 30%, 27%, 50%); }
I hope I could help you.😊
Marked as helpful0 - @kale-Dev1Submitted over 1 year ago@firoozehImanyPosted over 1 year ago
👋 Hi, Kale
🎉Congratulations, your solution looks great. 🎉
I want to tell you the things that come to my mind to improve the solution. If you apply this change to it, your solution will be better
CSS:
main { box-shadow: 5px 20px 50px 10px hsl(221deg 100% 96%); border-radius: 32px; } .result-box { width: 360px; padding: 36px; border-radius: 32px; } .result-box h1 { font-size: 1.5rem; } .circle { padding: 40px; } .circle .score { font-size: 4.5rem; } .circle p { /*There is no need to adjust the position.*/ } .summary-box { gap: 16px; padding: 36px; width: 360px; } .summary-box>div { padding: 12px; } .summary-box div.button { margin-top: 24px; }
I hope I could help you.😊
0 - @ani-menaphireSubmitted over 1 year ago
Any kind of feedback would be appreciated :))
@firoozehImanyPosted over 1 year ago👋 Hi, dear Ani
🎉Congratulations, your solution looks great. 🎉
I want to tell you the things that come to my mind to improve the solution. If you apply this change to it, your solution will be better
CSS:
.card { background-color: #fff; box-shadow: 0 20px 20px hsl(245deg 75% 62% / 20%); } .card__container { padding: 24px; background-color: hsl(225, 100%, 98%); border-radius: 10px; }
Marked as helpful0