#8605 closed bug (notabug)

Opened September 27, 2012 05:52PM UTC

Closed September 28, 2012 01:44AM UTC

Last modified September 28, 2012 01:49PM UTC

jQuery UI Download page missing a dependency for position plugin

Reported by: Myrlyn Owned by: jquery-infrastructure
Priority: minor Milestone: 1.9.0
Component: [meta] ui.downloadbuilder Version: 1.8.23
Keywords: Cc:
Blocked by: Blocking:
Description

I noticed the following bugfix that Scott Gonzalez added roughly three months ago:

https://github.com/jquery/jquery-ui/commit/98772fd0a1094f7fb2fbe1d8a95557bf2b545f6e

We realized this change today on one of our pages using the position plugin which uses the deprecated .curCss method. After finding Scott's fix in the jQuery UI core file, we realized that position now has a dependency upon core. Wanting to double check this dependency, I went over to the jQuery UI download page, and set it to do a custom download. I deselected all components, and then selected just the position component. Typically, if a component has a dependency, the other components it relies upon are automatically selected for the user. this didn't happen in this case. Am I correct that this is a bug with the download page now that position has this reliance upon core?

Thanks!

Keith

Attachments (0)
Change History (2)

Changed September 28, 2012 01:44AM UTC by scottgonzalez comment:1

resolution: → invalid
status: new → closed

I'm not sure why you think that bug fix makes position dependent on core; that would be quite silly. If you download just position from the download builder, it will work. Position handles missing $.curCSS on its own.

Changed September 28, 2012 01:49PM UTC by Myrlyn comment:2

Sorry, our version where we found the problem was 1.8.20. And I missed the correction in the current version. Thanks.