@someshwari-rudra
Submitted
Fell Free to provide your feedback..
Looking to hire developers?
@NaveenGumaste
@someshwari-rudra
Submitted
Fell Free to provide your feedback..
@NaveenGumaste
Posted
Hello Someshwari rudra ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Always use h1
first and then h2
, h3
and so on
Headings should only increase by one use h1 first then h2 etc..
If you use vs code the check this article
happy Coding😀
@niteshh-s
Submitted
I am new to Web development. I Made this website using beginner level HTML and CSS. No bootstrap used. It appears perfect on desktop screen. I would really appreciate if anyone can suggest me how can i make this website completely responsive.
@NaveenGumaste
Posted
Hello Nitesh Singh ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Always use h1
first and then h2
, h3
and so on
Headings should only increase by one use h1 first then h2 etc..
Warp your card content around the main tag Ex: 👇
<body>
<main class="container">
*all you content here*
</main>
</body>
If you use vs code the check this article
happy Coding😀
Marked as helpful
@ilhanhakan-koc
Submitted
Any comments or suggestions are welcome! :)
@NaveenGumaste
Posted
Hay ! Hako Good Job on challenge
Always use h1
first and then h2
, h3
and so on
Headings should only increase by one use h1 first then h2 etc..
If you use vs code then check this article
Keep up the good work!
Marked as helpful
@Yaser47
Submitted
Leave Any Advice
@NaveenGumaste
Posted
Hello Yaser47 ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Always use h1
first and then h2
, h3
and so on
Headings should only increase by one use h1 first then h2 etc..
happy Coding😀
Marked as helpful
@Nebil-Nej
Submitted
Like always, any and all comments are welcome.
@NaveenGumaste
Posted
Hello Nebil Nejmudin ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Payment button text is just white color.
annual plan bg color is given in the style-guide.md
happy Coding😀
@roshankcpkr
Submitted
Hoping for better suggestions
@NaveenGumaste
Posted
Hello Roshan K.C ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Add Main tag after body <main class="container"></main>
Always use the "alt attribute" and write what img is , and if the img is for decorative then leave it empty but always add it with alt.
happy Coding😀
@Blue-Cheesecake
Submitted
Is there anything needed to improve? Are there any suggestions for beginners?
@NaveenGumaste
Posted
Hello SINUT WATTANRPORN ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Add Main tag after body <main class="container"></main>
The background color is given in style-guide.md
Why us section color is too much light and i know it was not given in style-guide so use color-zilla chrome extension. and you will get the color
This is color i used for my solution --light-cyan: hsl(179, 45%, 52%);
check my solution here if needed
happy Coding😀
Marked as helpful
@nasimabhari
Submitted
This is the second project I've done on frontendmentor. I'll be glad to hear any helpful advices!
@NaveenGumaste
Posted
Hello Nasim Abhari ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Always use h1
first and then h2
, h3
and so on
happy Coding😀
Marked as helpful
@MarcusTuliusCiceron
Submitted
Going to as much challenge as possible in the coming weeks month, this the first one I complete.
@NaveenGumaste
Posted
Hello Ciceron ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Add Main tag after body <main class="container"></main>
happy Coding😀
@eeffoCdecI
Submitted
Hey everyone! I'm trying to become a front-end developer. This is my first attempt. Please let me know if you have any suggestions. Thank you!
@NaveenGumaste
Posted
Hello a coffee mug ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Backgound img should be added 'Css'
Add Main tag after body <main class="container"></main>
.
Always use h1
first and then h2
, h3
and so on.
happy Coding😀
Marked as helpful
@nuelobeto
Submitted
Hi guys! I would love to get your feedback on my code and ways to improve. Thanks.
@NaveenGumaste
Posted
Hello Nuel ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Always use h1
first and then h2
, h3
and so on
Add Main tag after body <main class="container"></main>
Consider using h2-h6
elements to add identifying headings to all sections.
happy Coding😀
@SAIKIRAN3670
Submitted
This is my first design using html and css. please provide your valuable feedback on this design
@NaveenGumaste
Posted
Hello Saikiran Rudra ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Add Main tag after body <main class="container"></main>
Always use h1
first and then h2
, h3
and so on
Headings should only increase by one use h1 first then h2 etc..
happy Coding😀
@airaciv
Submitted
Hello, this is my first challenge from this website to practice my HTML and CSS. Feedback and comments are also welcome. One question that I have about this preview card is about the hover state of the card image. I would like to have the view-icon to have higher opacity like how it was shown in the design. I've tried several methods, even looking through other solution, but I'm still confused on how they do it unless I copy paste the whole thing. I've tried using another div to contain the view-icon and use lower opacity for those container, while using opacity 1 for the view-icon but it still doesn't work for me. Increasing the z-index of the view-icon, in this case, also doesn't work. Either it really doesn't work this way or I'm coding it wrong. If anyone could please kindly explain it to me, I will be very thankful!
@NaveenGumaste
Posted
Hello airaciv ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
font-weight
to the "Eth" 0.41.happy Coding😀
Marked as helpful
@eeffoCdecI
Submitted
Feel free to feedback! Thank you!
@NaveenGumaste
Posted
Hello a coffee mug ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Always use h1
first and then h2
, h3
and so on
Headings should only increase by one use h1 first then h2 etc..
Add Main tag after body <main class="container"></main>
happy Coding😀
Marked as helpful
@khshakib
Submitted
Give Feedback
@NaveenGumaste
Posted
Hello Kamrul Hassan ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Add Main tag after body <main class="container"></main>
you cannot use "1000px" as you did screen and (max-device-width: 1000px)
happy Coding😀
@ibrahimano
Submitted
Hello world! It's been a long time since I submitted a challenge, and I'm excited to hear any feedback from you so feel free to write one. While working on this challenge I wasn't sure what to use with those four cards so at first, i tried css grid, but I couldn't arrange those four cards in the same way so I changed it to flexbox and yup finally It worked. My questions : should i have used the css grid? is it better than flexbox in this case. what do you think?
@NaveenGumaste
Posted
Hello ibrahim ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Add Main tag after body <main class="container"></main>
To center the card vertically
min-height: 100vh;
display: flex;
justify-content: center
align-item: center;
Always use h1
first and then h2
, h3
and so on
Headings should only increase by one use h1 first then h2 etc..
Consider using h2-h6
elements to add identifying headings to all sections.
Always use the "alt attribute" and write what img is , and if the img is for decorative then leave it empty but always add it with alt.
happy Coding😀
Marked as helpful
@M-Attiyah
Submitted
Let me know your opinion
@NaveenGumaste
Posted
Hello Mahmoud Attiyah ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
You have used wrong font-style
for the bottom numbers.
add img overlay as given in style-guide.md and opacity.
happy Coding😀
Marked as helpful
@boostbm
Submitted
Any feed back helps!!
@NaveenGumaste
Posted
Hello David Yu ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
happy Coding😀
Marked as helpful
@avldez
Submitted
Any feedback is welcome.
@NaveenGumaste
Posted
Hello Aldo ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Add font-weight
to the "eth" and its number.
Add Main tag after body <main class="container"></main>
- Check my article on accessibility issues
Always use h1
first and then h2
, h3
and so on
happy Coding😀
Marked as helpful
@SourceCodeDad
Submitted
Please provide any and all feedback.
Thank you! b
@NaveenGumaste
Posted
Hello Roque Rodriguez ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
<body>
<main class="container">
*all you content here*
</main>
</body>
see if you use <ul>
and <ol>
should have <li>
as there child. no other tag will work
happy Coding😀
@TomasCosta1
Submitted
I'm starting in this programming and this is my first project. Recommendations are accepted to change or optimize the code
@NaveenGumaste
Posted
Hello TomasCosta1 ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Add Main tag after body <main class="container"></main>
Consider using h2-h6
elements to add identifying headings to all sections.
happy Coding😀
@hteinLinn210
Submitted
I start using Flexbox and Media Queries. Any advices to improve my code are warmly welcome.
Happy coding!
@NaveenGumaste
Posted
Hay ! Htein Linn Good Job on challenge
Add Font-weight
to the numbers and increase the font-size
Have a good day and keep coding 👍!
Marked as helpful
@AlsoShantanuBorkar
Submitted
Don't have any questions but any advice is appreciated.
@NaveenGumaste
Posted
Hello ShantanuBorkar ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Title font is too big pls reduce it.
Always use h1
first and then h2
, h3
and so on.
-Warp your card content around the main tag Ex: 👇
<body>
<main class="container">
*all you content here*
</main>
</body>
happy Coding😀
Marked as helpful
@KamyaJain
Submitted
do check out if i had done the challenge correct or not.
@NaveenGumaste
Posted
Hello KamyaJain ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Add Main tag after body <main class="container"></main>
Always use h1
first and then h2
, h3
and so on
Headings should only increase by one use h1 first then h2 etc..
happy Coding😀
Marked as helpful