Skip to content
OverviewSolutions
25
Comments
4
Dabinder Udhan
@dabinderudhan

All comments

  • Dabinder Udhan•420
    @dabinderudhan

    Hi @catherineisonline, Amazing work and keep coding... cheers

  • Dabinder Udhan•420
    @dabinderudhan

    Hey @AlexManu05, good job completing this challenge. Keep up the good work.

    Your solution looks great however when one hover on the button, the whole content moves upward. This is because you have applied border:0 to your button and border: 1px while hovering the button. You can add border:1px solid transparent to your button. this will stop the movement of the content upwards when we hover the button.

    I hope this helps

    Cheers Happy coding 👍

    Marked as helpful
  • Dabinder Udhan•420
    @dabinderudhan

    Hi, Your solution is lacking background image in both desktop and mobile versions. Just add background image to your solution.

    Overall good job. Keep coding.

    Marked as helpful
  • Dabinder Udhan•420
    @dabinderudhan

    Your image tag should have "alt" attribute. And incase of icon and svg images you can leave the vale of the "alt" attribute blank and add the "aria-hidden=true".

    Rest you did a good work. Keep coding.

    Marked as helpful