Not Found
Not Found
Not Found
Oops! You need to be logged in to do that
Oops! You need to be logged in to do that
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Oops! You need to be logged in to do that
Not Found
Oops! You need to be logged in to do that
Not Found

Giang Nhật Khánh (Kayden)

@KaydenGiang2512
Vietnam
170Points

I'm an IT student at RMIT University trying to master the way of the code (and hopefully land a job in the future) :D

Latest Solutions

Latest Comments

  • Solution screenshot
    • HTML
    • CSS

    qr-card-component

    6
    Giang Nhật Khánh (Kayden)170 | Posted 17 days agocommented on Muneeb Ahmed's "QR code component" solution

    Hi Muneeb, it seems that your project is incomplete, as the QR image is missing from the page and only the text section is visible. Please check your code and fix any errors you may have made. Good luck!

    0
  • Solution screenshot
    • HTML
    • CSS

    Responsive NFT Preview Card incl. Transition

    2
    Giang Nhật Khánh (Kayden)170 | Posted 18 days agocommented on BjoernBennat's "NFT preview card component" solution

    Great work on this project Bjoern! I noticed only a single accessibility issue with your code (see the report section for more details) where your attribution <div> is not contained within any landmarks:

    <div class="attribution"> Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>. Coded by <a href="#">BjoernBennat</a>. </div>

    In order to fix this, you just need to change the <div> tag into a <footer> tag (a landmark element in HTML). Also, adding transitions to your text hover would make the aesthetics even better (by using the same transition you used for your image hover). Hope this helps and happy coding :D

    1
  • Solution screenshot
    • HTML
    • CSS

    Social Proof Section (Mobile First Design)

    2
    Giang Nhật Khánh (Kayden)170 | Posted 21 days agocommented on Kent O'Sullivan's "Social proof section" solution

    Brilliant work Kent! I just noticed a single accessibility issue in the report generated by Frontend Mentor, where there is no <h1> tag in your entire document. Other than that, your solution was very close in resemblance to the original design. One suggestion that I would make is to set up all your colors as custom properties to be used widely across your stylesheet simply by calling var(--your-color).

    1
  • Solution screenshot
    • HTML
    • CSS

    NFT Preview Card Solution

    8
    Giang Nhật Khánh (Kayden)170 | Posted 27 days agocommented on Lacey E's "NFT preview card component" solution

    It was probably because I didn't phrase my suggestion well enough (quite hard to do so through text-only comments!), so I will paste in the link to my solution here so you can see what I tried to explain above. Hope this clears up any confusion you may have on our issue!

    https://github.com/KaydenGiang2512/Article_Preview_Component

    Best of luck :D

    1
  • Solution screenshot
    • HTML
    • CSS

    NFT Preview Card Solution

    8
    Giang Nhật Khánh (Kayden)170 | Posted 28 days agocommented on Lacey E's "NFT preview card component" solution

    Hi Lacey, great work on this project overall. I actually completed this project a few weeks ago so I might have a solution to your "eye image opacity" issue, which is by separating the eye-icon from the NFT image into 2 separate <div> tags. After that, create another <div> as the overlay and use position: absolute for all of these to ensure that they stack on top of one another. By putting the eye-icon in its own <div> tag, the opacity attribute from the NFT image will no longer affect it, and you can independently set the opacity of the eye to your choosing. Hope this helps and happy coding!

    P.S. the image is not horizontally centered in the entire card element, so that's another thing you should look into.

    0