Since upgrading to 8.4.2 I have started getting errors in the console.
Uncaught TypeError: Cannot set property 'position' of undefined
at position-min.js?v=1.12.1:4It only seems to occur in the Frontend theme, not when viewing the Seven admin theme. I've tested with my custom theme as well as Stark and Bartik and still get the same issue. It only occurs when logged in. I managed to make the error disappear by removing all admin toolbar modules and the core toolbar module.
Has anyone else encountered this?
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 2925997-4.patch | 401 bytes | tikaszvince |
Comments
Comment #2
dtraft commentedI am also having this issue!
In my case, we have a limited admin toolbar for site editors and the error only occurs for them - it appears to work fine for site admins.
Comment #3
codesmithI have a custom theme and am getting this error when *not* logged in.
Comment #4
tikaszvince commentedThe code of
assets/vendor/jquery.ui/ui/position-min.jsdepends oncore/jquery.uilibrary, but the library definition ofjquery.ui.positiondoes not mark this dependency. So when other library depends oncore/jquery.ui.position(i.e. quickedit) they have to depends oncore/jquery.uitoo.The attached patch set
core/jquery.uias dependency forjquery.ui.positionlibrary.Comment #5
tikaszvince commentedComment #6
joelpittetNice catch, thanks @tikaszvince for the patch and @ndolley for the issue
Comment #7
codesmithPatch in #4 fixes the issue for me. Thank you @tikaszvince!
Comment #8
larowlan8.4.x will only receive critical backports now
Comment #10
catchCommitted/pushed to 8.6.x and cherry-picked to 8.5.x. Thanks!