Ticket #3228 (closed bug: fixed)

Opened 6 years ago

Last modified 6 years ago

Slider fire beforeunload event

Reported by: Cloudream Owned by: paul
Priority: major Milestone: 1.6rc2
Component: ui.slider Version: 1.5.2
Keywords: Cc:
Blocking: Blocked by:

Description

$(window).bind("beforeunload",function(){alert("fired");}); $("#slider").slider();

move handle will fire beforeunload in IE7

Change History

comment:1 Changed 6 years ago by Cloudream

href="javascript:void(0)" in handle cause this problem.

comment:2 Changed 6 years ago by paul

  • Status changed from new to closed
  • Resolution set to fixed

Fixed in r688.

comment:3 Changed 6 years ago by Cloudream

  • Status changed from closed to reopened
  • Resolution fixed deleted
  • Milestone changed from TBD to 1.6

I think it should be

.bind('click', function(e) { e.preventDefault(); })

but not

.bind('click', function() { return false; })

jQuery core fixes this event cross browsers.

comment:4 Changed 6 years ago by paul

  • Status changed from reopened to closed
  • Resolution set to fixed

Good idea, fixed

comment:5 Changed 6 years ago by rdworth

  • Milestone changed from 1.6 to 1.6rc2
Note: See TracTickets for help on using tickets.