Some of the repeating CSS values were successfully merged into a single class. But there are some CSS classes in the CSS file that still have repeating values that can be merged into a single class to adhere to the DRY principle.

There is some code that has been commented out in the CSS style which should be removed as this is perceived as bad practice in software development.

• it is good job. as optional you might good if you adapte how to write good documentaion and description.

• In the app.js file line num 19 the error p.js:19 Uncaught TypeError: Cannot read properties of null (reading 'addEventListener') at app.js:19:36 was fond in your javascript Dom manipulations.

• I would suggest that you change the Id name in the following code <div class="hamburger" id="open-nav" onclick="openNav()"> to open-view.

Modal view: Display image issues

Hello @RileyManda. Good job on your project so far. However, there are few issues that can be fixed to maked your project even cooler 😄.

The images currently have been well integrated. You don't see them because gave an image showing that a 'gray background'. I represented it by the red marker.
I suggest that you give a new image source (represented by the pink marker) to each of the objects in your workcards array if you want to have an image for each of your cards.

Remove the javascript events code from the HTML file.And utilize the app.js code to dynamically manipulate the dome.
TODO: Remove onClick from Html tags.

Modal view: Display image issues

Hello @RileyManda. Good job on your project so far. However, there are few issues that can be fixed to maked your project even cooler 😄.

As you can see in the screenshot below, you are actually importing the image from the workcards array instead of the detailscontent array.

I suggest that you create a loop that will take all the images of detailscontent array into a new array arrayX. You can then embed each of the images when you create your detailViewContainer

The Website icons can be replaced with fa font icons to improve the loading speeds of the website.Using images can cause slow loading speeds.

Empty strings in the object which is not a good practice. Kindly declare something within the string.

You have made some good improvements to your portfolio but some sections need modification.

• Display the current project image when clicking ( see project button )
• the header is well done from a design perspective but it is covering the content of the showcase from large screens
• (optional) Try to make the body a bit smooth on scrolling, for this one you can use the scroll behavior property