Skip to content
OverviewSolutions
30
Comments
10

Yuko Horita

@Sloth247Washington State, USA645 points

A self-taught web developer who is always brushing skills. Techstack: HTML, CSS (Scss/Sass), JavaScript, TypeScript, React, Next.js, Remix Tools: Git/GitHub, Firebase, WordPress

I’m currently learning...

Backend and database

Latest solutions

View more solutions

Latest comments

  • Yuko Horita•645
    @Sloth247

    Hi Michael,

    Nice work! I like the animation effects and the fancy favicon😁

    I have the following suggestions;

    Issues in Accessibility Report

    • <div> should not be inside <a> tag. You can check from this site
    • In Navbar.js, you include <li> inside <NavLink> however, <NavLink> should be inside <li> and <li> must be directly under <ul>. I think these will fix your issues in the report.

    I am sorry, I maybe a bit of OCD, I think the font weight of h2 in Destination.js needs to be 400 and give more letter-spacing. Also please give more margins/paddings between the planet choice buttons.

    I think the website will be much closer to the design if you utilise the figma file, as this is free+ challenge.

    I appreciate if you give some feedback to my solution too!

    Marked as helpful
  • Yuko Horita•645
    @Sloth247

    Hi JibbyCodes, this is the best solution that I've ever seen on FEM for this challenge. I may need to revise my coding based on your solution 😁 I'm sorry if I am wrong but the useEffect for the theme in your context file should be "useLayoutEffect". I did a bit of research after finishing my solution and according to official React documentation, "useLayoutEffect will be flushed synchronously, before the browser has a chance to paint.", so this hook might be good to use there.

    I appreciate if you give me feedback on my solution too 😊

    Your solution is really nice.

    Marked as helpful
  • Yuko Horita•645
    @Sloth247

    I like your animation, and would like to use GSAP to my future projects! You included my profile icon which is funny 😋

  • Yuko Horita•645
    @Sloth247

    This is just a question but why do you use form for the price card?

  • Yuko Horita•645
    @Sloth247

    It looks awesome! I like your banner on Github too. I also tried this challenge a while ago and could not close the panels by only HTML and CSS (without JS). I appreciate if you give me a feedback on mine too!

    Marked as helpful
  • Yuko Horita•645
    @Sloth247

    Hey, I have just done the same challenge, I achieved the similar design with the original in footer with psudo element ::after. Please see my solution if you would like! https://www.frontendmentor.io/solutions/meet-landing-page-w-sass-sa598Ef7n

View more comments