Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

md-slider dragging broken when jQuery loaded in 0.7.1 #1352

Closed
jelbourn opened this issue Jan 30, 2015 · 8 comments
Closed

md-slider dragging broken when jQuery loaded in 0.7.1 #1352

jelbourn opened this issue Jan 30, 2015 · 8 comments

Comments

@jelbourn
Copy link
Member

When jQuery is loaded on the page, dragging the slider with the mouse is broken.

http://jsbin.com/recepu/5/edit

@maxee
Copy link

maxee commented Feb 1, 2015

+1, Same problem here - Chrome 40 (Desktop) on Win7 (64) and 0.7.1

@ThomasBurleson ThomasBurleson added this to the 0.8.0-rc1 milestone Feb 1, 2015
@TheHippo
Copy link
Contributor

TheHippo commented Feb 3, 2015

It does work for me - Chrome 40, Linux. Using commit f1a4bcc.

@jelbourn jelbourn changed the title Can no longer drag md-slider with mouse as of 0.7.1 md-slider dragging broken when jQuery loaded in 0.7.1 Feb 3, 2015
@jelbourn
Copy link
Member Author

jelbourn commented Feb 3, 2015

Update: the issue only occurs when jQuery is loaded on the page.

@jelbourn
Copy link
Member Author

jelbourn commented Feb 3, 2015

Looks like this has to do with the dispatchEvent function in gesture.js. One of the users found that they could fix the issue by triggering the event to the jQuery-wrapped event target:

if (window['jQuery']) {
  window['jQuery'](eventPointer.target).trigger(eventObj);
}

@ajoslin
Copy link
Contributor

ajoslin commented Feb 3, 2015

There's a pull request open to fix this.
On Tue, Feb 3, 2015 at 11:20 Jeremy Elbourn notifications@github.com
wrote:

Looks like this has to do with the dispatchEvent function in gesture.js.
One of the users found that they could fix the issue by triggering the
event to the jQuery-wrapped event target:

if (window['jQuery']) {
window'jQuery'.trigger(eventObj);
}


Reply to this email directly or view it on GitHub
#1352 (comment).

@ThomasBurleson
Copy link
Contributor

@ajoslin - that PR looks good (with some tweaks). Are you planning on merging it or should I do that ?

@ajoslin ajoslin closed this as completed in 88813b7 Feb 4, 2015
@Ronani
Copy link

Ronani commented Nov 1, 2015

Hi, I still have this problem with jQuery 2.0.2 and the latest version - 1.0.0-rc2. any ideas?

@jelbourn
Copy link
Member Author

jelbourn commented Nov 2, 2015

@Ronani could you open a new issue with a reproduction?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants