@j-hutchisonSubmitted over 2 years ago
Any suggestions or possible improvements would be greatly appreciated!
omardavidcyber
@omardavidcyberAll comments
- @omardavidcyberPosted over 2 years ago
one improvement is the border radius. you seem to have them on all the sections. In the design only the first and last one have a border radius one the outside borders.
Marked as helpful0 - @grmbyrnSubmitted over 2 years agoWhat are you most proud of, and what would you do differently next time?
I made this a long time ago so don't remember.
What challenges did you encounter, and how did you overcome them?I made this a long time ago so don't remember.
What specific areas of your project would you like help with?I made this a long time ago so don't remember.
@omardavidcyberPosted over 2 years agoThe only nit-picky thing is the page background color is white while the design is a more grayish one. But other than that the layout looks really good Graeme!
Marked as helpful0 - @sauravchamoli17Submitted almost 5 years ago
I have used flexbox for the project and developed it in my laptop and it is smaller than the actual desktop design. Constructive criticism is most welcome!
@omardavidcyberPosted almost 5 years agoThe background image is not showing up because you need to change the url location of the image. You have this:
background: url("/images/bg-desktop.svg");
Its trying to look for it in the css directory so you need to move up one directory.
Change it to this:
background: url("../images/bg-desktop.svg");
1
