Skip to content
OverviewSolutions
4
Comments
4
BledanIT
@BledanIT

All comments

  • BledanIT•60
    @BledanITPosted about 12 hours ago

    The thing that really stands out is the different font format used throughout the page. I think it's because the fonts are not imported, you can do it with two lines in the html head section. Look for the style guide delivered with the challenge for more informations.

    0
  • BledanIT•60
    @BledanITPosted about 13 hours ago

    Almost perfect! There is only a tiny difference in size for name and location text but the rest is spot-on!

    0
  • BledanIT•60
    @BledanITPosted about 17 hours ago

    Very good solution! The only two problems I notice are that the card itself is a tiny bit smaller than the intended design, as well as the "Learning" box.

    Marked as helpful
    0
  • Obi-Etchu•10
    @Obi-EtchuSubmitted about 23 hours ago
    What are you most proud of, and what would you do differently next time?

    I completed the task

    What challenges did you encounter, and how did you overcome them?

    Hosting i researched

    What specific areas of your project would you like help with?

    I dont know yet

    QR code component

    1
    BledanIT•60
    @BledanITPosted about 21 hours ago

    The code is very short and condensed, which is a plus since it reflects optimization and readability. However I'm seeing two problems:

    • The card is placed at the beginning of the pace, while the assignment shows it in the center. This can be done through CSS by adding to the card rules "position:absolute, top: 50%, left: 50%" and "transform: translateX(-50%) translateY(-50%)".
    • The text overflows beyond the card's borders. this might be doable by adding a CSS rule "max-height: fit-content" to the card.
    0