Skip to main content
Code Review

Return to Answer

replaced http://stackoverflow.com/ with https://stackoverflow.com/
Source Link

OK, this code looks fine to me. The only thing I can suggest is that you might prefer to use event.preventDefault instead of return false in your event handlers, because in jQuery, return false in an event handler causes both event.preventDefault() and event.stopPropagation(), which you may not want.

References:

OK, this code looks fine to me. The only thing I can suggest is that you might prefer to use event.preventDefault instead of return false in your event handlers, because in jQuery, return false in an event handler causes both event.preventDefault() and event.stopPropagation(), which you may not want.

References:

OK, this code looks fine to me. The only thing I can suggest is that you might prefer to use event.preventDefault instead of return false in your event handlers, because in jQuery, return false in an event handler causes both event.preventDefault() and event.stopPropagation(), which you may not want.

References:

Source Link
kojiro
  • 1.8k
  • 1
  • 13
  • 22

OK, this code looks fine to me. The only thing I can suggest is that you might prefer to use event.preventDefault instead of return false in your event handlers, because in jQuery, return false in an event handler causes both event.preventDefault() and event.stopPropagation(), which you may not want.

References:

default

AltStyle によって変換されたページ (->オリジナル) /