Skip to content
OverviewSolutions
16
Comments
13
Michael
@mikester380

All comments

  • Michael•430
    @mikester380Posted over 2 years ago

    Great solution man. I love it!

    0
  • akaigyouhou•300
    @akaigyouhouSubmitted over 2 years ago

    There are two problems I've met. 1, Is it right to first determine the "width" and "height" of the "card", then add the "padding" or "margin" to flow the inner content in the right position? 2, To make the eye icon in the middle of the Equilibrium picture and also the cyan color layer when hovering the picture. I use two layer. Because the first one should not be opacity while the second one should be half opacity. Maybe there are better solution.

    Using React with pure CSS

    1
    Michael•430
    @mikester380Posted over 2 years ago

    Hey there. Congrats on completing this challenge. You don't have to set a height on the container as the margins and paddings you use as seen in the design will add up to create the overall height of the container.

    0
  • Michael•430
    @mikester380Posted over 2 years ago

    Hey coder! I love your solution. I think you should also hide the advice ID until the advice has finished loading before display it.

    0
  • Michael•430
    @mikester380Posted almost 3 years ago

    Hey man, great solution, I like it!

    1
  • Michael•430
    @mikester380Posted almost 3 years ago

    Hey! Your design looks pretty good except for the responsive part which needs more work. Apart from the that, you did a really good job. Congrats

    Marked as helpful
    0
  • Michael•430
    @mikester380Posted almost 3 years ago

    Your design looks dope man. You good!

    0
  • Adrian Hartanto•60
    @adrianh42Submitted almost 3 years ago
    • Is the HTML semantic correct?
    • After testing in Chrome's device emulation, it doesn't seem to scale the content in the way I'd like on certain devices, like a Galaxy Fold or smaller iPhones, maybe because of the lower dpi? Is there a way to work around this?

    Order Summary Component with SCSS and Flexbox

    1
    Michael•430
    @mikester380Posted almost 3 years ago

    Hey Adrian,

    Congrats on completing this challenge. I couldn't find any problems with your design. Good work! 👍

    0
  • Karishma Garg•1,010
    @karishma-devSubmitted almost 3 years ago

    Hey Guys, I just finished the Stats Preview Card Challenge. I would like you all to see it and let me know how I can improve it. Let me know my mistakes . Thank you!.

    Html, Css

    1
    Michael•430
    @mikester380Posted almost 3 years ago

    Hey Karishma,

    Your card is not properly centered. Try removing the margin on the card container and set the body element to display: flex; and justify-content: center; to properly center the card container.

    Apart from that, your solution is very impressive. Congratulations!

    Marked as helpful
    0
  • Morgan•100
    @morgan03Submitted about 3 years ago

    I found this challenge quite difficult especially with the 'expand image part' where the images expand with a dark background, how the basket changes position on mobile and also putting the icon arrows in a rounded background. Would appreciate some feedback to see how other people approached it!

    Ecommerce page created HTML, Flexbox, Bootstrap and Vanilla JS

    1
    Michael•430
    @mikester380Posted about 3 years ago

    Set a width on the container of the arrow icons and use border-radius: 50%; to make it a rounded circle

    Marked as helpful
    1
  • Michael•430
    @mikester380Posted about 3 years ago

    Hey there, congrats on completing this challenge.

    I recommend you using "background-position-y: center" on the input image to center it vertically in the input field.

    And for the result section, I think you should add the dollar symbol to get it close to the design as possible.

    0
  • Aidan Wright•30
    @greasypicadilloSubmitted about 3 years ago

    Hello All,

    Hope you are well. This is by no means a pixel perfect recreation but I did have some fun with it.

    Some glaring problems for me: The share icon button looks a tad elliptical instead of nice and perfectly round Share icon does not change color properly The tooltip on the desktop design does not properly toggle off (this one really bugged me because I can't figure what changes to make to my script to make it function properly. Should be under the showDesktopSocial() if you can tell me where I went wrong.

    Also would love some tips for Unobtrusive Javascript! read a bit about it and want to implement on future projects.

    Thanks so much! I really appreciate anyone reading this and replying with constructive criticism!

    Best Regards, Aidan Wright

    Mobil-First Design with CSS Flexbox/HTML5/JS

    1
    Michael•430
    @mikester380Posted about 3 years ago

    Hey Aidan, congratulations on completing this challenge.

    You can try setting a width and height on the share icon and using border-radius: 50%; on it to make it a perfect circle.

    Also, try reducing the height of the component so it can match the given design and use flexbox to center it vertically on the screen.

    And as for the tool tip you complained about, I was unable to view your source code on github and give you a possible solution. Try fixing that so I can try my best to help you

    0
  • Alicja•30
    @amstelmachSubmitted about 3 years ago

    Hello! This is my first challenge with Frontend Mentor. Any feedback & suggestions would be highly appreciated! :)

    How readable my code is? How can I improve readability of my code ?

    I've tried to match the design as close as possible - I think I've managed to do so, but please judge it by yourself!

    Thanks, Alicja

    room homepage challenge completed with html, css & vanilla js

    2
    Michael•430
    @mikester380Posted about 3 years ago

    Hey there, good job on completing your first challenge. I've previewed the website and discovered it's not responsive, try fixing it so as to work on smaller devices.

    0