AYUSH RANJAN
@aayusranjanAll comments
- @Geo0510Submitted over 2 years ago@aayusranjanPosted over 2 years ago
Hey @Geo0510! I went through your solution, it's good.
-
I would suggest you try to avoid use
</div>`div
, for example you use<footer>
instead of `<div class="attribution"> Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank" >Frontend Mentor</a >. Coded by <a href="#">Esteban Salas</a>. -
you can look to at my solution as a reference
Marked as helpful0 -
- @SantiRomero27Submitted almost 3 years ago
Any suggestions?
@aayusranjanPosted almost 3 years agoGood work, Santiago!
- use
main
instead of<div id="root">
0 - use
- @YoThatsCrazySubmitted almost 3 years ago
This is my very first time coding and I got stuck for a bit but I stuck with it. Github is throwing me through a loop but I'm having trouble with Github pages and my images showing on my live URL. Let me know what I can improve on even though this is basic, and help on the Github pages as well.
@aayusranjanPosted almost 3 years agothe reason is not showing your image is:
- you used your local path, instead of
<img class="card-img" src="C:\Users\erica\OneDrive\Desktop\QR Code practice\images\image-qr-code.png" alt="">
use<img class="card-img" src="images/image-qr-code.png" alt="qr code image">
.
Marked as helpful0 - you used your local path, instead of
- @emanuelstromSubmitted almost 3 years ago@aayusranjanPosted almost 3 years ago
Great work Emanuel !
- There will you can add hover on
.item__button
and.checkout__pay-btn
it will be more better. - You don't need to creat button tag, it's like so good to go with a tag in
.item__button
and.checkout__cancel-btn
you can check out my solution
Marked as helpful1 - There will you can add hover on
- @fabifeliciaSubmitted almost 3 years ago
I would like feedbacks regarding the css code and semantic html tags
@aayusranjanPosted almost 3 years agoThat's really great work. I would like to give some feedback
- Ig no need of section and id selectors (most case avoid to use id selectors ).
- add root in CSS ( it's would really helpful, and add there all color given in style guide)
- change paragraph color. ( it might be better if you use class in h and p tag )
you can go through my solution : https://www.frontendmentor.io/solutions/qr-code-component-aAq0MFl4r
Marked as helpful2 - @ThermonnSubmitted almost 3 years ago
Why don't the svg icons show up in the live site?
@aayusranjanPosted almost 3 years agoadd img tag before <h2 class="heading">Sedans</h2>
0