@Bluorde
Submitted
@CaplexW
@Bluorde
Submitted
@CaplexW
Posted
Not bad! But I'd like to point out one thing that can improve your pages in the future:
Try to avoid using px
unit at padding, margin, font-size, ... etc. It will limit resonsivness because it's an absolute unit. Instead you can use responsive units like em
or rem
.
You can dive deeper in this topic by reading this article.
@nerywst
Submitted
What are you most proud of, and what would you do differently next time?
I was very glad that I was able to make a very similar design to the exercise.
@CaplexW
Posted
Good job!
Though, you probably forget to make an 'active-state' of design to buttons. You can do it using pseudo-class :hover
, but I see you already know that trick.
You can also improve resposivnes by avoiding using absolute unit like px
and instead using responsive unit like rem
or em
.
Marked as helpful
@hieutran249
Submitted
@CaplexW
Posted
It's a good job! But I'd like to highlight a few things that might become an issue in the future:
px
. It's an absolute unit, so it may cause problems in a responsive design. Instead, you can use responsive units like rem
or em
. You can read more about this in this article: Why you shouldn't use pixels for font-size.vh
and vw
when you use them in height
and width
. It might sound good to display content depending on a screen's height and width; however, it might act unpredictably while the screen size changes. If you open your page in DevTools and play with the screen width, you'll see that the layout acts probably not like you expected it to. Giving a .container
property height: 100%;
(100% from parent height) and using responsive unit like rem
to declare elements size, should solve the problem with changing screen size.What are you most proud of, and what would you do differently next time?
I was able to use CSS to center the component
What challenges did you encounter, and how did you overcome them?
Initially I couldn't vertically align the QR component, but after learning about flex box , I was able to do it.
@CaplexW
Posted
Good job! I see nothing to add.