Skip to content
OverviewSolutions
8
Comments
16
seyide hundeyin
@SeyideHundeyin

All comments

  • Vjekoslav•485
    @Vj3koSubmitted almost 4 years ago

    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

    Huddle landing page with a single introductory section

    3
    seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

    Hi Vjekoslav,

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

    It's also responsive!!

    Have a nice day!

    0
  • Mark Sainsbury•45
    @Sa1nersSubmitted almost 4 years ago

    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.

    Social Proof Section - Desktop only

    1
    seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

    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
    @Vj3koSubmitted almost 4 years ago

    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 !

    Single grid mobile first using grid

    2
    seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

    Looks really nice and responsive!

    1
  • seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

    Hi Gautam,

    Congrats on your first challenge.

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

    Have a nice day

    0
  • Gildardo Rosas•190
    @shkr1Submitted almost 4 years ago

    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. 🤟

    Fylo landing page site HTML CSS GRID BEM FLEX

    1
    seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

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

    I recommend fixing the report issues.

    Have a nice day!

    0
  • Jesús Alejandro Atao•970
    @JesusAtao96Submitted almost 4 years ago

    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 😊.

    Easybank landing page with CSS Grid

    3
    seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

    This is very nice and it matches the design perfectly.

    Happy coding!!

    1
  • seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

    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
  • seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

    Hi,

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

    0
  • seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

    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-bwSubmitted almost 4 years ago

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

    Html, CSS and Flexbox

    1
    seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

    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
  • seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

    Look good

    0
  • seyide hundeyin•440
    @SeyideHundeyinPosted almost 4 years ago

    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