Use more correct boudning rectangle in SparkOverlay.pointInOverlay() + nits

TBR: @terrylucas

In particular, this fixes the problem where a click on a <button> or other plain HTML element in spark-overlay would close it (with autoCloseDisabled == false), such as the now-gone + and - buttons to switch themes and shortcuts.