Hmmm, I like everything up to this bit. In all other themes, $secondary-color is a background color, so that's what I'd assume when I first look at the code.

How about calling it $inactive-color instead, or similar?