If we have places where a `selector` variable is an actual jQuery element, we should change that.  A selector should always be a string.  The problem I have with the `$foo` syntax is that it is confusing.  Let me explain.  In jQuery using code you get used to `$` being at the beginning of expressions, and when it is, it is actually doing something - accessing the jQuery namespace to call a function = it always transforms what follows.  When I saw the `$element`, I thought to myself, "hmm, OK what jQuery *function* is being called here."  It took me a few minutes to look around at the context and see that it was just a plain old variable name, thinking, "wait, that must be a typo, what is going on here."  If *I* get that confused by something, other people are going to trip over this...Also, even after I figured out what was going on, I stumbled over the other instances of this.  So, I do think we should not use this naming convention in our code.