[patch] [ccla] ScrollableView does not work with dojo-1.4

[ykami]

It works fine with dojo-1.3 and dojo-1.5, but not with dojo-1.4. Mixed-in methods seem to be missing.

[Douglas Hays]

Patch from Yoshiroh Kamiyama (IBM, CCLA) to avoid an inheritance problem on dojo 1.4 (#10709 and #10788)

[Douglas Hays]

While this patch is simple and doesn't even directly affect 1.6 functionality, I think this patch needs to be reviewed/committed by Jared and/or Shane since it is adding special code to work with dojo 1.4. Apparently the dojox/mobile code is being used by some customers who either cannot or will not move up to 1.6 in the near future but who require mobile capabilities. Besides adding a small amount of code bloat (and why not fix the underlying dojo 1.4 problem?), supporting customers on older dojo versions will lock dojox/mobile into non-AMD style module definitions and non-HTML5 compliant test files.

[Shane O'Sullivan]

It would be better to not hard code in the exact version to match. How about checking just that the minor version is less than 1.5?

Doug, how about we leave this as it is for 1.6, which is the last version the 1.4 guys can use, and fix it in a point release to use the new system?

[Jared Jurkiewicz]

I agree with Shane. We leave it in non-AMD format for 1.6, then switch it over post 1.6 to be AMD and get rid of the check.

[Jared Jurkiewicz]

It also sounds like the issue he's working around was 1.4 specific, so the check it's doing for exactly that is probably fine.

[Douglas Hays]

(In [23383]) Fixes #12001. Proxy commit for ykami (IBM, CCLA). Workaround Dojo 1.4 specific bug using explicit version check (an alternative bug detection patch was found to be not feasible).