SpiderOak/SpiderOakMobileClient — pull request #245 by kenmanheimer (2013-05-21 17:24:50+00:00)
Title: Make the entire Remember Me settings entry active, and have it trigger a

click on the checkbox, to have the existing change handling code do its
job.

Fixes #229.

I have a small question regarding this change.

I'm adding an event.preventDefault() to both the new rememberMeSetting_tapHandler() and the already existing rememberMe_changeHandler(). I believe, however, that the latter will never be necessary, because the active encompassing anchor, together with the .preventDefault() in the .rememberMeSetting_tapHandler(), will only get the click via the .trigger('click') from the method associated with the anchor, never directly from the UI. Should I leave out the second .preventDefault()? Does it matter, one way or the other?