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

  • Namrata Sinha• 530

    @nmrtsnh

    Submitted

    Hello everyone,

    I am Namrata and this is my solution for the Expenses chart component challenge.

    If you have any suggestions to improve my code, please feel free to share!

    Thank you 🙏

    Expenses chart component

    1

    enzmrg 💫• 890

    @enzo-mir

    Posted

    Hi ! I love the way that you make the hover effect on the chart's elements and the design is really nicely made ! Just the responsive could be improve without any breakpoint with a min-width of 100% should be good and try to avoid margin to positionning ;)

    I loved the solution !

    0
  • enzmrg 💫• 890

    @enzo-mir

    Posted

    Hi solution ! Maybe should be nicer if the mobil breakpoint would be at E.g : 600 px of viewport because 375px is a bit small at my sense.

    Well done ;)

    0
  • enzmrg 💫• 890

    @enzo-mir

    Posted

    Background color fixed :)

    0
  • Agnik Bakshi• 480

    @Agnik7

    Submitted

    Hi everyone, This is my solution for the Rest Countries API with Color Theme Switcher challenge.

    I have used ReactJs, TailwindCSS and Material Icons to finish this project, and have tried to stick to the design as much as possible.

    Rest Countries API with Color Theme Switcher

    1

    enzmrg 💫• 890

    @enzo-mir

    Posted

    Hello ! Awesome solution but something went wrong when you try to get details of some countries

    • E.g : American Samoa

    Try to fix it for a fully respected challange !! Well done ;)

    Marked as helpful

    1
  • Gift Richard• 520

    @fibonacci001

    Submitted

    OMG you guys, I just finished another solo React project building this gorgeous Easybank landing page and can I get a high five?! 🖐️ Got those slick animations going, components breaking things into reusable bits - I'm basically a pro-React developer now! 😎

    But let's get real - accessibility was kinda my weak point if I'm honest. 🤓 I was nerding out hard about toggle interactions and hover states when I should've been asking: but what about screen reader users?!

    So I've gotta work on my inclusivity game for sho! Here's some ways I can have MORE fun (!) while making sites that work for errbody:

    🔤 Semantic HTML is my BFF (but like cooler than my actual BFF maybe?)

    🎨 Ooh baby those color contrasts are SPICY and accessible!!🌶️

    📽 Alt text for images - now screen readers can enjoy my glorious creations too!

    There's always more to learn when it comes to web access I know, but I'm so jazzed to keep improving. Because the web is for EVERYONE, am I right or am I right?? 🙌

    My code organization is getting bonafide lit too - scoping CSS by component ftw! Less clashes than high school, woohoo! 💃 Sure I can tweak some things to share styles globally but I've come a long way from one messy CSS file, phew! 😅

    So let me know if you've got tips for me my coding queens and kings! ✨ And high five to more inclusive and kick-butt websites for all users!! 🖐️🖐️ Raised hands AND hand clap emoji for the people in the back!! 🙌👏

    an animated react banking landing page

    1

    enzmrg 💫• 890

    @enzo-mir

    Posted

    Awesome project the only things that i can say :

    • There is a lack of spacing between elements and line spacing
    • And careful to tabindex because the application is not really accessible be keyboard

    Well done this solution deserve compliments !

    1
  • Ulrich1979• 160

    @Ulrich1979

    Submitted

    This was a good exercise, i loved the design of this site. Nevertheless it took me a few hours. Your feedback or any suggestions to improve my code is very welcome.

    Insure landing page

    1

    enzmrg 💫• 890

    @enzo-mir

    Posted

    Hello that is a very nice solution the only issue is the footer doesn't look like the design :

    • There is no background color
    • The shape doesn't appear this way

    And concerning your code : Nothing to say you have sementics tags maybe in JavaScript :

    • Declare readable contants such as "top_nav" instead of "x"
    • Use classList to improve the code it will be better with classList.add(), classList.remove() or classList.toggle()

    Well done ;)

    0
  • enzmrg 💫• 890

    @enzo-mir

    Posted

    Awesome challenge ! Just to be fair the footer/header does not match with the design in mobil view else if nice responsive web app.

    Well done ;)

    Marked as helpful

    0
  • Shakeel Ahmad• 60

    @Shakeel-Coder

    Submitted

    i tried my best as i am the beginner in the front-end development. but i miss some serious issues such as email validation before submit and back from success message. i tried hard but i didn't know how to do this. so please if you have submitted this solution and know how to do this give me the some suggestion about that. i hope you check and give me the feedbacks.

    signup with success message

    2

    enzmrg 💫• 890

    @enzo-mir

    Posted

    Hello there, nice challenge ! There are a lot of things to say but firstly :

    • Try to avoid div element and use instead html sementics tag
    • Your parents elements : body, div#main got no height to be the container and you can not easily center the main content
    • Set min-width to your containers it will help for responsive

    I guess that is your first FrontEnd Mentor project ? Well donne ! ;)

    Marked as helpful

    0
  • cwatso• 70

    @cwatso

    Submitted

    I have used flexbox to center this vertically by setting the height to 100vh, but I am curious if there is a more widely accepted way of doing this? Would love some feedback.

    QR Code Component using HTML and SASS

    2

    enzmrg 💫• 890

    @enzo-mir

    Posted

    Hello ! nice challenge ! To avoid the 100vh on your body try this :

    html,body {width : 100%; height:100%}; body {display : grid; place-items : center}

    And that's it you have a centered container ! Well done ;)

    1
  • enzmrg 💫• 890

    @enzo-mir

    Posted

    Hello nice work !!

    I did same as you in the first place concerning le accrodion that you can open only one but this doesn't work like that you have to add a way to toggle the asks. And careful at your responsive, your button trim the content. Theses little mistakes are ignorable, that's just for the perfection ;)

    Well done !

    Marked as helpful

    0
  • enzmrg 💫• 890

    @enzo-mir

    Posted

    Awesome challenge !! Just one thing to be boring ;) :

    • The mobil nav bar get an "black transparency" at be background of the rest try to implement that !

    Marked as helpful

    0
  • enzmrg 💫• 890

    @enzo-mir

    Posted

    Hello ! nice solution jus be careful at you breakpoint between 768px and 1440px, your svg at the second section make and horizontal scroll cause of his width I guess, try to fix it !

    1