Skip to content
OverviewSolutions
7
Comments
4
P
D3V
@d3vaqa

All comments

  • P
    D3V•120
    @d3vaqa

    well done. just the height looks so different compared to the original design. if you could adjust the height. it will be looking much nicer. thanks

    Marked as helpful
  • P
    D3V•120
    @d3vaqa

    well done, just checkout the accessibility issues as they are pretty important

  • P
    D3V•120
    @d3vaqa

    Good job! in terms of accessibility there are some issues that I have noticed

    1 - instated of <div class="cuadro-interno"> it would be much better to use <main class="cuadro-interno"> 2 - in <img src="" alt ="" > in this case its necessary to fill the alteration for your image as => <img src="" alt ="QR Code" > 3 -change <p class="texto-uno"> to <h1 class="texto-uno"> 4 - chuck the <div class="attribution"> class outside your <div class="cuadro-interno"> and put it inside footer tag.

    thanks

    Marked as helpful
  • P
    D3V•120
    @d3vaqa

    thanks @Shashree Samuel, I will definitely update that.