Skip to content
OverviewSolutions
8
Comments
4
Javier Lopez
@jglopezre

All comments

  • Javier Lopez•145
    @jglopezre

    Hi, congrats, nice work, you could improve images sizing, using the object-fix CSS property for cutting images to cover entire box, maintaining sizes and ratio. You could use anchors for nav links, social links and other, and using :hover selector on these for make a pointing effect, like a color changing, scaling, or something else. You must use style guide where is how look in active mode. Greetings

  • Javier Lopez•145
    @jglopezre

    Hi, Congratulations... You could change color highlight text on text boxes, there are in style-guide.md. And, please, don´t forget making README.md page, with live link, we are able to test responsive and interactive behavior of the page.

    Greetings.

    Marked as helpful
  • Javier Lopez•145
    @jglopezre

    Congrats.. I suggest to you, at footer, in social links, you have to wrap every <img> into an anchor <a>, for give to icons the possibility to send user to respective page, and you could give to <a> margin-left and margin.right to open space between. Optionally, you could give to these new <a> option to hoverize (a:hover) giving interactive response when user pass mouse pointer over a icon.

    Greetings.

    Marked as helpful
  • Javier Lopez•145
    @jglopezre

    Nice, I am going to test it, I like use media query for background changing, but, i hadn't thought in it. Thank.