Skip to content
OverviewSolutions
7
Comments
34

eagermonument81

@ixtk210 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

View more solutions

Latest comments

  • eagermonument81•210
    @ixtkPosted over 2 years ago

    Hey, looks like the images are being stretched. They lose aspect ratio. Avoid setting width directly, use max-width. Right now you can apply object-fit rule on the images. Look it up!

    0
  • eagermonument81•210
    @ixtkPosted over 3 years ago

    body::before creates horizontal scrollbar on big monitor screens. Also, it appears that if I hover buttons on left or right, it affects the rest of them by adding 1px border

    Marked as helpful
    1
  • eagermonument81•210
    @ixtkPosted over 3 years ago

    I wouldn't recommend using wv unit for font-sizes. The problem is that if user changes their font size in settings, your website doesn't adapt. It also doesn't change font size if user zooms in/out. So it is only responsive to width of the window, which is not ideal. On some widths, font looks way too big/small (743px, 764px)

    Other than that, consider adding main element and check the report

    Marked as helpful
    0
  • eagermonument81•210
    @ixtkPosted over 3 years ago
    • there's no h1 on the website, consider using it for "Group Chat for Everyone" text
    • .intro__title is not centered on desktop layout
    • there's horizontal scrollbar at about 800px
    • button text font size is smaller compared to the layout. Consider adding font-size: inherit
    • I think h2 font-sizes are too large for mobile layout
    • you could add main element in html to indicate which part of the page is the main content

    There are other layout issues like images not being centered on mobile screen or top images messing up layout. Make sure to use browser dev tools to inspect how different elements/layout will look on different screen sizes. If you have specific questions feel free to ask

    0
  • eagermonument81•210
    @ixtkPosted over 3 years ago

    The website says "loading". It doesn't seem to work

    0
  • eagermonument81•210
    @ixtkPosted over 3 years ago

    You can remove the padding and add display: grid and place-items: center to center the text in the circle. Give the bottom one white background color.

    There's horizontal scrollbar at about 1200px

    Marked as helpful
    0
View more comments