@artuino0
Latest comments
- @yallsobad
You should add some Javascript to make it responsive. Minimum you could have it clear all the red dots, background color and notification counter at the top when "Mark all as read" is clicked.
Styling looks good tho!
Marked as helpful - @paulaabro@yallsobad
Hey!
I would remove the height on the container so when you expand all the questions the container height increases as needed.
You could also put cursor:pointer under the summary element so it works on the entire element instead of just the text.
- @Jey223@yallsobad
Hi!
-I put 'margin: 0 padding: 0' in the * class at the top of the css. It's a good global reset. -Try putting a max-width on your overall container (.div--holder) with 'margin: 0 auto' to keep things tight and centered on large screens. -Add the hover properties to the social icons same as the button
- @DrMESAZIM@yallsobad
Make sure to import and use the correct font. Put a container around your content with a max-width and 'margin: 0 auto'. It looks good on 1440px width, but above that it spans out.
- @Lwmeek@yallsobad
Hi! Looks pretty good!
On my monitor the image cuts off at the bottom, so I would put a max width somewhere to prevent that.
I would put a container around your content with a max size and 'margin 0 auto' to center it
ex:
<main> <section class=container>or just <main class=container>
Marked as helpful - @MariusHor@yallsobad
Looks good! I like the fonts in :root I'm going to start using that.
In the code you could:
Use * {} to remove the default margins, instead of referencing every tag I used grid for the main container instead of flex
