BEM intruder

I have seen some code that was really consistent in the naming of the classes, but then out of nowhere there is some modifier class, like nav--black.
My two cents are to be consistent in the CSS architecture and in naming your classes.

Background shorthand property

Order of background shorthand and background non shorthand property is important.
My two cents are to split that shorthand into multiple properties.