Skip to content
Submitted over 1 year ago

Result summary component using gird and media query

jeffreylaichunghang•60
@jeffreylaichunghang
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Results summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

The difficult part is the media query. It looks like that I have to redefine the sizes, width and height all over again. My css looks like a mess, not quite readable. I am still figuring out a way to reduce the use of class.

Community feedback

  • Abdul Khaliq 🚀•72,560
    @0xabdulkhaliqPosted over 1 year ago

    Hello there 👋. Congratulations on successfully completing the challenge! 🎉

    • I have other recommendations regarding your code that I believe will be of great interest to you.

    POINTING CURSOR ↗️:

    • Looks like the component's button element has not a pointer, this property plays a major-role in terms of both UI & UX
    • The cursor: pointer CSS property is important for button-like elements because it changes the cursor from the default arrow to a pointer when hovering over the element. This provides a visual cue to the user that the element is clickable and encourages interaction.
    • In terms of UI/UX, using cursor: pointer helps to improve the usability of the interface by making it easier for users to identify interactive elements. It also helps to provide feedback to the user by indicating which elements are clickable and which are not.
    • So we want to add this property to the following button element
    button {
cursor: pointer;
}
    • Now your component's button has got the pointer & you learned about this property as well

    .

    I hope you find this helpful 😄 Above all, the solution you submitted is great !

    Happy coding!

    Marked as helpful
    0
  • _nehal💎•6,710
    @NehalSahu8055Posted over 1 year ago

    Hello Coder 👋.

    Congratulations on successfully completing the challenge! 🎉

    Few suggestions regarding design.

    • Use gradient button:hover

    • background-image: linear-gradient(#6542FE,#342CE2);

    • Use max-width instead of width to make your design responsive

    • Use cursor:pointer for buttons for more user friendly.

    • Use Semantics for the proper design of your code.

    <body>
<header>
<nav>...</nav>
</header>
<main>...</main>
<footer>...</footer>
</body>

    I hope you find this helpful.

    Happy coding😄

    Marked as helpful
    0
  • John Mirage•1,590
    @john-miragePosted over 1 year ago

    Hello, did you wrote the css from mobile to desktop ?, if not it's maybe a better idea to go mobile first, it should be easier and more readable (and it's a best practice because of the mobile devices performances)

    0
    jeffreylaichunghang•60
    @jeffreylaichunghangPosted over 1 year ago

    Indeed. I kind of feeling the same way when I finish it. Can you tell me more about why it is beneficial to mobile devices performances please?@john-mirage

    0
    John Mirage•1,590
    @john-miragePosted over 1 year ago

    @jeffreylaichunghang When writing desktop first, a mobile device has to load the desktop styles by default and after the mobile styles in the media queries so all the styles is loaded. When mobile first you load the mobile styles by default but not the desktop styles (because of the media queries). So a mobile device has to load less styles when writting mobile first.

    Marked as helpful
    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord