Kamania
@KamaniaAll comments
- @Aladdin-ElshamySubmitted about 1 year ago@KamaniaPosted about 1 year ago
Great Job Aladdin. I have also completed the task a few moment ago. One thing I have noted, try set the selected rating as light grey for the background color instead of orange. Use orange when hovering over the rate buttons.
1 - @BintangRPSubmitted over 1 year ago
- i was difficult on writing css best practice and dont know the order of syntax like
.container { object-fit: cover; display: flex; flex-direction: row; margin: 5px width: 500px height: 1000px }
I still don't understand the preparation of the best practice.
it's my first try on frontendmentor.
@KamaniaPosted over 1 year agoJob well done @BintangRP, just noted that the background color is different from what is provided on the design. I suggest add the following in your css
html, body{ background-color: hsl(212, 45%, 89%); }
0 - @SomHsSubmitted over 1 year ago
Feedback Welcome! I did this project using VW as a measure because I feel commutable using it but your opinion is highly valuable. Thanks!
@KamaniaPosted over 1 year agoHello @MatiasFrascino, add this code
border-top-right-radius: 0;
to remove the border radius on the top right. This will help your solution be similar to what we have on the design.Marked as helpful0 - @winstr00mSubmitted over 1 year ago
How did you make this page responsive with clean code, my code is no where near clean.
@KamaniaPosted over 1 year agoHello @Winstroom, on the cart button, add a
margin-bottom: 1em;
to help add a space from the bottom border0 - @pouripzSubmitted over 1 year ago
Hello, I have a problem with the background of the site. What would you do if you wanted to design the background to look like this project? How do you place the pattern? Thank you for helping me
@KamaniaPosted over 1 year agoI experienced the same issue. I temporarily resolved it by using Internal CSS. Insert the background image in the css internally
0