Skip to content
OverviewSolutions
27
Comments
29
P
enzmrg 💫
@enzo-mir

All comments

  • Namrata Sinha•530
    @nmrtsnhSubmitted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    Background color fixed :)

    0
  • Agnik Bakshi•480
    @Agnik7Submitted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    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
    @fibonacci001Submitted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    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
    @Ulrich1979Submitted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    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-CoderSubmitted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    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
    @cwatsoSubmitted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    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 💫•920
    @enzo-mirPosted 12 months ago

    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