elhawlir/prj-rev-bwfs-dasmoto

prj-rev-bwfs-dasmoto's Issues

Suggestion - no need to specify font-weight: bold on h1 and h2

Seen here: https://github.com/elhawlir/prj-rev-bwfs-dasmoto/blob/master/Dasmoto's%20Arts%20and%20Crafts/resources/style.css#L5

Since this is bolded by default, there's no need to specify it here.

Suggestion - remove duplicate stylings of the font-family

Seen here: https://github.com/elhawlir/prj-rev-bwfs-dasmoto/blob/master/Dasmoto's%20Arts%20and%20Crafts/resources/style.css#L3

Since these are already set up (via the html selector on line 1), there's no need to specify it again here.

Nice job noticing the similarities between the h2s and using one general styling

Seen here: https://github.com/elhawlir/prj-rev-bwfs-dasmoto/blob/master/Dasmoto's%20Arts%20and%20Crafts/resources/style.css#L10-L15

I really like what you've done here. Well done! :)

SUMMARY: Meets Expectations

Sweet job on this project! I really liked your use of a general styling in your code, and how clean and easy to read your code was. In terms of improvement, I recommend removing the font-weight: bold on your h1s and h2s, using spans in your HTML, and reducing duplicate stylings in your CSS.

If you’re in the mood for a challenge...I would look into adding more items to each section and finding a nice way to lay them out. Here's a link to get you started - https://www.w3schools.com/css/css_rwd_grid.asp. Keep up the great work! ^_^

Suggestion - instead of using strong tags, I suggest using spans

Seen here: https://github.com/elhawlir/prj-rev-bwfs-dasmoto/blob/master/Dasmoto's%20Arts%20and%20Crafts/index.html#L17

Since the strong tag is a stylistic one (i.e. it bolds the content inside) I recommend using spans and applying a font-weight: bold on the span instead. This way, we can keep all our styling in our CSS (one place).