Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • Vjekoslav 485

    @Vj3ko

    Submitted

    hi, just finished this project. i did it but i feel that i am doing something wrong, like there are so simpler solutions, and i complicate things. maybe if someone can take a look at the code and give me some guidance. i would appreciate it

    @SeyideHundeyin

    Posted

    Hi Vjekoslav,

    This looks very nice and matches perfectly with the design file.

    It's also responsive!!

    Have a nice day!

    0
  • @Sa1ners

    Submitted

    My first time using Grid, so probably some mistakes. I am sure that there is a more efficient way to create 5 stars! I tried using the image repeat function within a separate div but then struggled to get the alignment correct so reverted back to multiple img entries in the html file.

    As ever, all feedback, suggestions for improvement welcome.

    @SeyideHundeyin

    Posted

    Hi Mark,

    I recommend you trying to make it responsive on mobile devices and position the second background image closer to the testimonial cards.

    Also, you can try to use flexbox or grid for this challenge as it will make it easier.

    Have a nice day!

    1
  • Vjekoslav 485

    @Vj3ko

    Submitted

    Finished this single price grid layout, i would be really happy if someone can check it out and comment what i can do better, or to shorten code, or anything really. Until next time, keep on coding !

    @SeyideHundeyin

    Posted

    Looks really nice and responsive!

    1
  • @SeyideHundeyin

    Posted

    Hi Gautam,

    Congrats on your first challenge.

    I recommend changing the background color to match the design file.

    Have a nice day

    0
  • @shkr1

    Submitted

    Hi everyone 👋

    I implemented this challenge using a subset or my understanding of BEM and minimal CSS with Grid and Flex when it would make sense. Also I tried to apply semantic HTML.

    The spacing and font sizes are a little bit off because I don't use the figma/sketch files 😅.

    Any feedback is much appreciated. 🤟

    @SeyideHundeyin

    Posted

    This looks very nice and responsive on a lot of devices.

    I recommend fixing the report issues.

    Have a nice day!

    0
  • @JesusAtao96

    Submitted

    Hi all 👋

    I decided to do this challenge with SASS, CSS Grid, Flexbox, BEM and a little animations.

    Bonus:

    • Grid Tracks names for layout.
    • Mobile nav only with CSS.
    • JS only to hide scroll when mobile navigation is open.
    • Background sections using css grid positioning only for testing purpose.

    Any feedback is appreciated.

    Happy coding 😊.

    @SeyideHundeyin

    Posted

    This is very nice and it matches the design perfectly.

    Happy coding!!

    1
  • @SeyideHundeyin

    Posted

    Hi Dina,

    I would recommend you making this responsive on mobile devices and changing the background color to match the design file.

    Have a nice day!

    0
  • @SeyideHundeyin

    Posted

    Hi,

    The font size on desktop is so small and it's difficult for me to read.

    0
  • @SeyideHundeyin

    Posted

    Hi Shreya,

    I would recommend using CSS grid for this challenge and also stick with using h1-h6 without jumping from h1 to h3.

    Also on mobile, it doesn't sit in the middle you can use "margin: 0 auto;" to centralize your container.

    You can learn more about CSS grid here: https://www.youtube.com/watch?v=jV8B24rSN5o&ab_channel=TraversyMedia

    Hope this helps!!

    Keep coding and happy coding!!

    1
  • Leon 180

    @leon-bw

    Submitted

    Open to any suggestions for improvement, I do believe there are some things that I can improve with this submission. Thanks

    @SeyideHundeyin

    Posted

    Hi Leon,

    This looks good and responsive on most devices.

    Also, check the accessibility and HTML issues report.

    Hope this helps.

    Keep coding and happy coding!!

    1
  • @SeyideHundeyin

    Posted

    Look good

    0
  • @SeyideHundeyin

    Posted

    Hi,

    You did a great job on this challenge and you can work on the media queries to make it more responsive on mobile.

    2
  • Mustapha 40

    @Shileex

    Submitted

    I'm just a beginner, I tried my best making it more responsive if I can get a guide on how I could improve more I will appreciate.

    @SeyideHundeyin

    Posted

    try using CSS grid for this challenge. it makes it way easier!

    0
  • Zach Kyman 220

    @zky63

    Submitted

    Could not get the span with "$29" inside to go back to 100% opacity even though I put the CSS for it after the CSS for the <p> element it is inside of.

    @SeyideHundeyin

    Posted

    Nice job.

    For the "$29" make it an h2 then span "Per month" so you can style them differently.

    also, I recommend changing the background color and add the border-radius as in the design.

    Hope this helps!

    1
  • P

    @mickoymouse

    Submitted

    Feel free to comment on what I should do to improve my code base. I'm currently diving in web development so correcting me or giving me instructions on what are the best practices is highly appreciated. Thank you!

    @SeyideHundeyin

    Posted

    Your solution is nice.

    I would recommend increasing the h6 and using max-width for the h6 so it can break into two columns.

    1
  • @SeyideHundeyin

    Posted

    nice job and it looks good on mobile 👍

    Also, check the tablet version and I can't find your code.

    1
  • @lucas-moont

    Submitted

    Finished this after a day or two. Learned a lot of things and still wanna add some features to it using JS. Anyway, here's how it's done to this day and probably I'll update it soon. Any tips or feedbacks are welcome and appreciated.

    @SeyideHundeyin

    Posted

    Hi,

    I cant view your solution.

    0