Design comparison
Solution retrospective
Hi,
This is my second project made with frontend mentor and I am very satisfied.
Do you think I could improve something? Write, comment. I will accept any criticism :)
Community feedback
- @NaveenGumastePosted almost 3 years ago
Hay ! Daniel Good Job on challenge
These below mentioned tricks will help you remove any Accessibility Issues
-> Add Main tag after body
<main class="container"></main>
-> Learn more on accessibility issues
Keep up the good work!
Marked as helpful1@xBuzAxPosted almost 3 years agoHello, @Crazimonk
Thank you for your feedback ;) I will definitely go to the reading which and study.
1 - @karishma-devPosted almost 3 years ago
Hi @xBuzAx, Your solution looks good and responsive.
Things that can be improved:
1.In desktop layout, instead of specifying an exact width of the card, use
width: fit-content
. So, in future , you have to add some content then you won't have to change the width of the card.2.In desktop layout, in
.desc-middle
class change yourmargin-bottom
to 2em from 4em. I saw that after changing this, your design looked better.- There are some lines of code in css that were not changing anything in your solution, like:
- In
section
:paddinng: 0 2em
,.desc
:align-items: center
, etc. Try to see these lines and remove them, so there is no extra code that is not modifying your solution in any way.
Hope this helps you. Good luck!
Marked as helpful1@xBuzAxPosted almost 3 years agoHello @Krrish105,
Thank you very much for your objective opinion and for your guidance. I will definitely use them and try to keep them in mind.
I am very grateful.
0 - @perezjprz19Posted almost 3 years ago
The color for the attribution is illegible, so you might want to consider changing the color since they would not meet accessibility standards.
I would recommend using opacity and mix-blend-mode to get the color of the overlay closer to the design.
Marked as helpful1@xBuzAxPosted almost 3 years agoHi @perezjprz19,
Thank you very much for your objective opinion and for your guidance. I was just wondering how I could get the intended effect regarding the darker purple photo.
I will definitely take advantage :)
Thank you for you suggestion.
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord