P
@waldekglaz
Latest solutions
Latest comments
- @dorianarri@Holllyyyy
Great job @dorianarri :3 You can add box-shadow if u would like it.
- @wqais@Holllyyyy
Hello @wqais, great job, well done! :3 I would recommend you for large screens to use *{paddin n margin 0, box-sizing border box} that will be applied on all ur elements... One tip: try to always make a folder for CSS, and inside that folder to put CSS files, it's much easier for other devs to reading your code. :3
Marked as helpful - @Apprentixx@Holllyyyy
Great job, be careful about accessibility issues... Always be sure to include <main> instead u putting div as a first step. Also, every page should have h1 first then h2, etc... On small devices, I would need to move left-right to find where is it, n on the larger screen, it's not centered. I would suggest you flexbox for centering, and to avoid positions here. Try to use %, rem, em instead of px, cause px are absolute lengths... Good luck.
- @RaiIsNotYourGuy@Holllyyyy
Great job! As you can see around ur picture, there's a little blue color, so put the background color white. :3
- @a2uuz@Holllyyyy
Hello, great job! :3 You can change h3 to h1 cause of accessibility, also <div class="parent"> should be actually <main class="parent"> also cause of accessibility so you will remove accessibility issues. You can add some box shadow for better design.
Marked as helpful
