@a3-1811
Submitted
My first time got challenge form FrontendMentor!
Looking to hire developers?
@koalalikecode
@a3-1811
Submitted
My first time got challenge form FrontendMentor!
@koalalikecode
Posted
Hey Nguyá»…n Quang Huy, congrats on finishing your first challenge! It looks so close to the design, but I think there is something you can do to improve your page.
a:hover{ color:...;}
Aside from that, good job on this challenge!
@Shahab-Malikk
Submitted
Any kind of feedback will be appericiated
@koalalikecode
Posted
Great work on this challenge bro, but I think you can have some regulations for a better job
background-size: contain;
Happy coding!
@Pablo-cyber21
Submitted
My first challenge on frontend mentor. All advice and critic are very much welcome. Qn. -How can I align the music svg icon with the rest of the elements on the Annual pricing section ?
@koalalikecode
Posted
Hey Asiimawe, congrats on your first challenge. I think you have grasped a basic use of HTML and CSS, but you need to practice more on your next challenge.
max-width: ...;background-color: white;
and have your img in it and the container class with max-width: 100%
. You can try these codes.background-repeat: none;
to remove the default value.Happy coding!
Marked as helpful
@mmc1999
Submitted
I have two problems that I cannot solve, please, I need recommendations:
@koalalikecode
Posted
Hey, great work on this challenge!
Aside from that, great work!
@opplayz
Submitted
Hi, I'm new to HTML and CSS. I know that my code isn't practical. If you can, please give me some suggestions on how I can clean up my code. Thanks!
@koalalikecode
Posted
Congrats on finishing your challenge. It looks good for a beginner.
However, you can improve your code by taking some steps:
<center>
, you had better use a main tag like this `<main>your codes</main><img class="musicicon" src="assets/icon-music.svg" alt="Music Icon" width="50%" height="50%">
<br> </br>
Aside from that, great work!
@ara6i
Submitted
give me some feedback pleas :) it's my first project and i need your opinion :)
@koalalikecode
Posted
Congrats for your first time, it looks so close to the design!
You can have some improvements for your code:
<h2>Order Summary</h2>
You had better use you h tag respectively such as h1>h2>h3 and CSS them.<button class="button pay">Proceed to Payment</button>
your Payment should be a descendant of a tag because it can link to another pagebackground-position: contain;
. It will work well on a larger screenAside from those, great work!
@Swan-Yee
Submitted
All Of your feedback and suggestion are welcome. I hope you will like my code. Happy coding :-)
@koalalikecode
Posted
It's a great solution. And I think it will be better if you use the main tag in this code:
<div class="main-container">
for make the landmark clearly.
And your background-image can use this for 100vw:
main-container { background-size: cover;(work with contain too) }
@Briancarlo24
Submitted
How was my solution?
@koalalikecode
Posted
I think you should add the hover state of this site
Marked as helpful
@TGURUTEJA
Submitted
I have a doubt, that how to find dimensions of elements by just seeing images?
@koalalikecode
Posted
I think one of the aims of these challenges is to help us practice seeing dimensions without tools
@DevmianS
Submitted
First "bigger" JS project, what would you change?
@koalalikecode
Posted
Great work, so close to the design! It will be perfect if some things change:
@Jakeyikapp
Submitted
Please let me know if there is anyway I can structure my HTML or CSS better or if there are any better ways to code this challenge.
@koalalikecode
Posted
Great work, so close to design. But you should use your h tag respectively h1>h2>h3...
Marked as helpful
@DeboraBrum
Submitted
I'm learning so any feedbacks are welcome! Thank you.
@koalalikecode
Posted
I think you should add the feature that when someone calculates several numbers respectively, he/she doesn't need to click the '=' button.
@T3sultan
Submitted
This frontend challenges awsome and practices...thanks
@koalalikecode
Posted
I think you use the wrong color for "Proceed to Payment" and you should add the hover states