Skip to content
OverviewSolutions
23
Comments
3
Wilou Nam
@Wilouricard

All comments

  • Wilou Nam•320
    @WilouricardPosted about 2 years ago

    Thank you for your help :) I just made changes

    Wilou

    0
  • Wilou Nam•320
    @WilouricardPosted about 2 years ago

    Hello! Great work Szymon!

    To add background to the card, you can:

    • Wrap all card's element in one parent div (if it is not done yet),
    • Set an ID to this parent div,
    • Go to your CSS file and custom the 'background-color' to the right color, also add some padding and border-radius

    Cheers :)

    0
  • saurabh•200
    @akgec-saurabhSubmitted about 2 years ago

    I am still learning JavaScript I wanted to try whatever I have learnt till now and I will surely update this when I finish my JavaScript.

    Frontend Mentor | Advice generator app

    1
    Wilou Nam•320
    @WilouricardPosted about 2 years ago

    Hello ! You're doing a great job :)

    I noticed that you used hover state on the 'dice' button to add the box shadow. The hover state for this button is a very slight colour change, and the box shadow should be used for the active state

    .dice-container:hover {
  /*background-color: ; */
}

.dice-container:active {
  box-shadow: 0 0 20px 5px var(--neon-green);
}

    Cheers

    Marked as helpful
    0