@sirnet
Latest comments
- @csimomelian
Hey looks clean and great,
I've noticed that there are an uncontrolled layout from 766px to 1024px that cause horizontal scroll because there are some fixed measurements. Fix it and it will looks perfect.
Keep coding dude.
- @imbhaskarn@csimomelian
Looks great dude! Keep coding, you are in the right way!
Only take a look to the html issues :D
- @ardaberrun@csimomelian
Yep, looks good. Any challenge, no matter how simple it may seem, has its complications.
You can increase the
line-heightvalue to allow the text breath more, and don't forget to define the alts in the images, is a good pratice. - @olgak169@csimomelian
The animations are great.
I would try to avoid using pixel units in favor of rem or em so that it can be more scalable and accessible.
The text bubbles instead of setting fixed width would be more convenient to use maximum width.
Rest, good job
- @georgelowe@csimomelian
The height of the content is higher than the height of the body, where normally it should forces the overflow scroll but by having
align-item:centerorjustify-content:centerthe flex-items do not cause overflow scroll.You can figure out with
min-height:100vhinstead ofheight:100vh.
