-
Notifications
You must be signed in to change notification settings - Fork 0
Description
Looks great! Your images are all working properly, your alert works properly. Some notes that do not require attention/resubmission, but just wanted to share for your own knowledge:
-
Usually all image files/graphics are kept in an image folder within your project so your files are organized for easy reference. If you continue with this project to keep it as your professional portfolio, I would add an image folder to your project and keep any jpg/png files in there. You can even have folders within that folder to separate your professional photos from your brick/background ones too.
-
Be careful when using inline styling and a style file together. In your html, your brick background placeholders for photos/links that are located in your flex container (lines 69-76) have height and width specified inline, but you have a .css file that handles most of your stylings. There is a hierarchy to styling, see this link for more details, so if you go to change something on your site in your .css file, but it doesn't update, be sure to check for inline styles that may be overriding what you're trying to do.
Keep up the good work!