Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Front-End Editing Broken — jQuery 3.1 #2033

Open
TomS- opened this issue Sep 11, 2016 · 6 comments
Open

Front-End Editing Broken — jQuery 3.1 #2033

TomS- opened this issue Sep 11, 2016 · 6 comments

Comments

@TomS-
Copy link

TomS- commented Sep 11, 2016

As per title, the front-end editing modal box is broken when I'm using jQuery version 3.10.

Javascript error given: Uncaught TypeError: f.getClientRects is not a function
Appearance: http://imgur.com/6OJfcDI

Edit: Downgrading to 2.2.4 fixes this

@adrianbj
Copy link

If you need jQuery 3.x, have you tried the migrate plugin: http://code.jquery.com/jquery-migrate-3.0.0.js

I think the error you are getting is due to jqueryui not yet being compatible with jquery 3.x

https://blog.jquery.com/2016/06/09/jquery-3-0-final-released/

While most things will work, there are a few issues that jQuery UI and jQuery Mobile will be addressing in upcoming releases. If you find an issue, keep in mind that it may already be addressed upstream and using the jQuery Migrate 3.0 plugin should fix it. Expect releases soon.

@TomS- TomS- closed this as completed Sep 12, 2016
@adrianbj
Copy link

Not sure that this really should be closed - I think it's a good reminder that jQueryUI needs updating when a new version that supports jQuery 3.x is made available. And to be honest I haven't checked that this is definitely the issue.

@TomS- TomS- reopened this Sep 15, 2016
@ryancramerdesign
Copy link
Owner

I don't think there's anything we can do about this at present since we rely on jQuery UI for this. But it sounds like longer term we should consider not using jQuery UI at all for this purpose, and perhaps add-in our own tiny modal window library to handle the front-end editor case.

@TomS- TomS- changed the title Front-End Editing Broken 3.0.33 Front-End Editing Broken — jQuery 3.1 Sep 22, 2016
@cb2004
Copy link

cb2004 commented Sep 22, 2016

If UI is the problem with jQuery 3.1 then UI has been updated to 1.12.1. Might be worth having a quick look.

@teppokoivula
Copy link

teppokoivula commented Nov 17, 2018

Just for the record: I haven't actually tested it, but based on the package.json file I would assume that jQuery UI now supports jQuery 3.x.

	"dependencies": {
		"jquery": ">=1.7.0 <4.0.0"
	},

@teppokoivula
Copy link

But it sounds like longer term we should consider not using jQuery UI at all for this purpose, and perhaps add-in our own tiny modal window library to handle the front-end editor case.

Definitely agreed. I think it's a bit much to require jQuery just to open a simple modal window. Definitely a good use case for either one of the smaller modal libraries, or some custom JS :)

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

No branches or pull requests

5 participants