Skip to content
OverviewSolutions
8
Comments
2
Omer7_7
@omerkhan7210

All comments

  • Omer7_7•100
    @omerkhan7210Posted about 2 years ago

    HELLO SHAFIQUL ISLAM’S, First of all you can use css variables instead of individual classes as this makes your code hard to understand for other designers, variables can be used to store any type of value and you can use them and change them globally in your css code as well, the second thing i noticed that the breakpoint of your design was above 1100px which is not the standard breakpoint for mobile widths, you can make it 768px and make your design at mobile-width first then design it on the desktop screens. Hope this helps brother!

    0
  • baxtro•20
    @baxtroSubmitted about 2 years ago

    I tried flexbox with this one. I'm not sure if I implemented it correctly but I gave it a whirl. I did most of this in the Atom code editor, and the hover state did not alter how it looked in the browser, but now that it's live and uploaded to the web, the button hover state changes the appearance of the content a bit. I'm not sure why. I used a CSS button generator, so I'm not sure if it's the code in there. Thanks for any feedback.

    Product Preview Card Challenge using html and css with flexbox

    1
    Omer7_7•100
    @omerkhan7210Posted about 2 years ago

    I think the button should have a cursor:pointer on it so that it would look interactive, secondly if you have used display:flex on your bigcontainer then you should use flex-basis:50%; on both of the child elements of the container and also on the text column as well where you have used width.You used flex-direction so you can also use gap property to give space between your text elements. Hope you will find this helpful! :)

    Marked as helpful
    0