Hello!! please send feedback for points of improvement. Thank for all. Bye!!!!
girldocode
@girldocode
All comments
- girldocode• 260
@girldocode
Posted
overall it looks very nice. just add border-radius to the perfume section.
1 - Zeus Tenerife• 80
@zakura1x
Submitted
I'm not sure if its fully correct or not
girldocode• 260@girldocode
Posted
it looks really nice. just apply border-radius to the image.
Marked as helpful
1 - kamalesh87• 10
@kamalesh87
Submitted
i ve tired to do but i thought by seeing others code would help me learn more techinical knowledge so i just posted it do give me the feedbacks!!
girldocode• 260@girldocode
Posted
Actually you did a good job , if you want to improve more ,you can try using these in your code:
=> use flex property set to center to bring that circle in middle.
=> use border radius on card.
=> give margin + padding .
=> use linear gradient.
=> use opacity to lightens the colors on summary section.
0 - girldocode• 260
@girldocode
Submitted
girldocode• 260@girldocode
Posted
thanks for the review . i will apply this in my upcoming projects.
0 - Mennatallah Hesham• 1,490
@Mennatallah-Hisham
Submitted
Hello, Frontend Mentor community. This is my solution for the Qr-Code Component challenge
Today marks the beginning of the Newbie Challenge, a unique opportunity for me to offer solutions for HTML/CSS challenges specifically designed for newcomers. Throughout this challenge, I aim to provide comprehensive guidance to fellow developers by adhering to industry best practices. By focusing on key areas such as
- Semantic HTML
- Clean web page structure
- CSS enhancements
- Accessibility
- Performance Optimization,
I aim to address common mistakes observed in the challenges submitted by other participants. Through this initiative, I hope to not only solve newbie-level projects but also share valuable insights and tips to aid in their learning journey.
In this challenge, I focused on reducing the load time of Google fonts to improve page performance by 880ms. you can check this article
This solution is tested by
if you want me to review your code, just comment (the challenge name + the areas you want me to focus on) and I will provide a review as soon as possible.
girldocode• 260@girldocode
Posted
Thanks for sharing great article. It is helpful resource to know about web accessibility. Your font CSS is missing color and size. Did you notice?
0 - barani19• 10
@barani19
Submitted
girldocode• 260@girldocode
Posted
You can add font family to improve text appearance. Overall looks nice.
0 - BenAli• 60
@chitos07
Submitted
girldocode• 260@girldocode
Posted
Overall looks nice. Just add margin around card. And also check font family.
1 - Nemanja Unković• 90
@nunkovic
Submitted
girldocode• 260@girldocode
Posted
Nice. You have used CSS nesting, it is good practice. You just need to add little margin in around card in mobile version.
Marked as helpful
1 - FurkanUcar7• 60
@FurkanUcar7
Submitted
girldocode• 260@girldocode
Posted
Looks beautiful, but need little focus on media query. At some point image does not show in mobile version.,
0 - KNaingrW• 70
@KNaingrW
Submitted
What did you find difficult while building the project?: Achieving precise alignment and spacing of elements in smaller mobile views was challenging. Ensuring a consistent and visually appealing layout across different screen sizes required careful adjustments.
Which areas of your code are you unsure of?: While I have tested the code extensively, I'm always cautious about cross-browser compatibility and responsiveness, especially on older browsers. Additionally, I would benefit from feedback on the overall code structure and organization to ensure it follows best practices and is maintainable.
girldocode• 260@girldocode
Posted
You can improve few things
- Background colors
- Sizing
- Fonts
0 - Y0USEFsobhy• 250
@Y0USEFsobhy
Submitted
girldocode• 260@girldocode
Posted
Looks nice but you need to improve few things like colors, fonts, and some spacing issues.
Marked as helpful
0
