Skip to content
OverviewSolutions
22
Comments
7

HaDo Dev

@hadodevSpain310 points

Self-taught Full Stack Developer (HTML, CSS, BOOTSTRAP, TAILWINDCSS, JAVASCRIPT, VUE.JS, PYTHON, DJANGO, MySQL, PostgreSQL, and learning more )

I’m currently learning...

Modern CSS | Modern Javascript | Large Language Models and IA development

Latest solutions

View more solutions

Latest comments

  • HaDo Dev•310
    @hadodev

    Hi @BiswajitHemram,

    Your desktop solution look great. These are my recomendations:

    • Don't forget the active states. You must add the :hover and :focus-visible to the learn more links/buttons. For example:
    button:hover,
button:focus-visible {
  background-color: white;
}
    • Pay attention to the responsive design. Mobile-first workflow is a good approach and it's widely use nowadays. You start your design in a mobile screen and after you add media queries @media or container queries @container to adjust the design for wider screens. In your desgin, you need to change the border-radius property of the first and the last sections.

    Keep going and happy coding!

    Marked as helpful
  • HaDo Dev•310
    @hadodev

    Hello @VISHALKANNAN070!

    Your solution looks nice and almost like the design. My recommendations:

    • I think it's an issue about the font. In your style.css file you've used this statement: @import url( https://fonts.google.com/specimen/Poppins;); which has a bug. Try the following: @import url('https://fonts.googleapis.com/css2?family=Poppins:wght@200;400;600&display=swap');
    • The h6 included in <div class='header'> doesn't much the design width. Try to use max-width: 50ch;(update the number to your requirement).
    • Try to achieve better spaces between the elements inside the four cards.

    Keep going!

  • Bhushan-Chaudhari•110
    @bhushu0910
    What are you most proud of, and what would you do differently next time?

    The implementation of the CSS part was easy and was proud of it

    What challenges did you encounter, and how did you overcome them?

    styling div class and div ID was a little confusing but solved it

    Frontend Mentor | Product preview card component

    1
    HaDo Dev•310
    @hadodev

    Hi @bhushu0910,

    Nice solution. Well done!

    My recomendations:

    • Try to acheive better the space between the elements. The <div class="right"> must have a wider padding-inline or margin-inline.
    • The image must have border-radius in the left side (desktop design).
    • I think that the main goal of the challenge is get a responsive layout. Your solution doesn't adapt for small devices.

    For responsive design, check out this ressource: 1-Line Layouts - 10 Modern CSS layout and sizing techniques that highlight just how robust and impactful a single-line of styling code can be.

    Keep going!!

  • HaDo Dev•310
    @hadodev

    Hi,

    Nice solution. Recomendations:

    • Try to choose the correct color for body background and the most of the text.
    • Also try to acheive better the space between elements.

    Keep going :-)

  • HaDo Dev•310
    @hadodev

    Hi @sunilthapa01!

    It's a nice solution. Here you have some recommendations:

    • For accesibility issues, try to use better contrast color on button hover. As a ressource you can use to check the contrast color, check out colors.io.

    Keep going :-)

    Marked as helpful
  • HaDo Dev•310
    @hadodev

    Hi @Grahamocean!

    • Nice pulse animation.
    • Try to acheive better the space between the elements.
    • Also pay attention to the font weight of the author. It's must be higher.
    • For the Learning label you might give it a better style. Here is an example:
    .label {
        display: block;
        width: 12ch;
        padding: .5rem;

        font-weight: var(--fw-bold);
        font-size: .875rem;
        text-align: center;

        background-color: var(--clr-yellow);
        border-radius: .25rem;
        padding-inline: .75rem;
}

    The custom properties used at :root are the following:

    --fw-bold: 800;
--clr-yellow: hsl(47, 88%, 63%);
    • For a better accesibility, use rem unit instead of px for fonts, border-radius, padding, etc...

    Good job! Keep going :-)

    Marked as helpful
View more comments