Skip to content
Submitted about 2 years ago

Order summary

Rony dos Santos Oliveira•60
@OliveiraRony
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Order summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

My first attempt at this challenge. I still have a little doubt when to use rem or px. Help me with this, please. I also ask for any feedback for my solution.

Community feedback

  • Lucas 👾•104,400
    @correlucasPosted about 2 years ago

    👾Fala Rony, beleza? Parabéns pelo desafio!

    Acabei de dar uma olhada aqui e ficou muito bom seu componente. A única coisa que você poderia melhorar seria puxar o background versão mobile, porque está cortando um pouco depois que a tela começa a diminuir, o melhor jeito de fazer isso é aplicando uma media query pra fazer a troca de background em base ao dispositivo.

    Parabéns e continue no foco!

    Marked as helpful
    1
  • enochlee•600
    @iamenochleePosted about 2 years ago

    Your solution is almost pixel perfect 👏, great job, you should use rem always, but I do use px for border radius most times.

    Marked as helpful
    1
    Rony dos Santos Oliveira•60
    @OliveiraRonyPosted about 2 years ago

    @iamenochlee thanks, buddy. ;D

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord