@mattreee
AliAbuhumra
@aliabuhumraAll comments
- @aliabuhumra
This Blog is Perfect
- @Kornazda@aliabuhumra
This card is perfect
- @Oswalld
- P@LandonRGeorge@aliabuhumra
This project is good but you have to practice postion in css to make background Easy
- @kowall1013@aliabuhumra
This Pod Request Perfect Wow .💞 There is a small note remove height : 100% in body
- @Zeighnab@aliabuhumra
Great job, but I have a few notes
1 - add background-color : hsl(277, 64%, 61%); to <div class="img"/>
2- add this styles for img tag .img{ background-color : hsl(277, 64%, 61%); } .img img { display: block; mix-blend-mode: multiply; opacity: 0.8; }
Marked as helpful - @nareshpoonia@aliabuhumra
Great job, but I have a few notes
1 - remove the tag < a herf=""> from inside button
2 - add landmarks in img like this 👉 </img>
- @ApplePieGiraffe@aliabuhumra
What is the benefit of your situation tag (label) , Then you hid tag lavel display :none;, Is it because of Accessibility or because of another?
- @ApplePieGiraffe@aliabuhumra
How do you know the names of the colors in :root, is there a specific site?
- @Nova988@aliabuhumra
amazing card
- @tanikalaw@aliabuhumra
1 - use semantic HTML5 , don't use only div 2- I loved your writing of BEM
- @Tuason066@aliabuhumra
Use main instead of <section class ="component"> to <main class="component"> because SEO browser
- @AgataLiberska@aliabuhumra
What I liked the most about your project is your use of em and rem amazing like you prtty
- P@palgramming@aliabuhumra
why don't use em or rem to padding and margins ????
