Skip to content
OverviewSolutions
13
Comments
3

Glozsa

@GlozsaMexico180 points

Learning front-end programming :D

Latest solutions

View more solutions

Latest comments

  • Glozsa•180
    @GlozsaPosted about 2 years ago

    Hi there!

    Some suggestions that will make your site look better

    .tagline

    instead of using:

    position: absolute; bottom: 10%;

    instead add:

    padding: 2rem 0;

    this will avoid your text to collide with the product card

    .card-image

    You can avoid using:

    object-fit: ;

    media (max-width: 675px)

    body

    you can avoid using:

    padding: 1.1rem;

    and if you fill like it, you can twik the scale of the body in order for it to be responsive with the smallest smartphones:

    scale: 0.85;

    semantic

    you should contain all contents inside a <main>

    Marked as helpful
    0
  • Glozsa•180
    @GlozsaPosted about 2 years ago

    Hi there!

    Some suggestions that will make your site look better

    body

    in your body selector on css (body)

    you can change the size of the background to: background-size: auto;

    This way it won't have any problems when scalating

    .icon

    Under your icons (.icon)

    You can put the transition inside the .icon instead of the .icon:hover

    This way it'll animate when the hovering stops

    .register-button

    Apply the same logic to the register button

    Marked as helpful
    0
  • Glozsa•180
    @GlozsaPosted over 2 years ago

    Hello there @TayAki79!

    Just a quick observation, the overlay image when you hover the NFT should have a border-radius. and your <HTML> should have lang="en" inside of it.

    Hope you find this helpful, happy codding and holidays :D

    Marked as helpful
    1