the active class is dynamic, so making it replace using regex would take more code as we'd need to make a new RegExp object. 

My view is that both classes should be dynamic. I am less opinionated on the use of regex, but my feeling is that we should use the smaller code footprint. If we're sticking with dynamic classes, that would mean string look ups.