dojox.image.Magnifier doesn't seem to work

[Enzo]

I'm using this magnifier on dojo 1.6.1. It works with a local copy of dojo, but doesn't work if I change to dojo CDN

Further more, both following links give js errors: (and magnifier doesn't work at all) ​http://archive.dojotoolkit.org/nightly/dojotoolkit/dojox/image/tests/test_Magnifier.html

​http://archive.dojotoolkit.org/nightly/dojotoolkit/dojox/image/tests/test_MagnifierLite.html

But if I go to a previous version of dojo, it works online:

​http://archive.dojotoolkit.org/dojo-2011-02-01/dojotoolkit/dojox/image/tests/test_Magnifier.html

​http://archive.dojotoolkit.org/dojo-2011-02-01/dojotoolkit/dojox/image/tests/test_MagnifierLite.html

[Adam Peller]

Looks like multiple bugs reported as one: CDN failure on 1.6, failure on nightlies (likely AMD migration and missing images)

[Enzo]

Well yes maybe 2 bugs reported into one, but maybe first bug comes from the fact I need to load dojo.js AND also Magnifier.js, in order to be able to do dojo.require('dojox.image.Magnifier'), so maybe this first bug would be vanished as soon as AMD is supported ? dunno...

I'll be able to make further tests asap AMD is supported indeed

[Adam Peller]

In [26496]:

AMD baseless conversion. Refs #13868

[Enzo]

Thanks gonna check that

But the file dojox/image/tests/images/spanke.jpg should be added to the changeset also, no? (it exists for ​http://archive.dojotoolkit.org/dojo-2011-02-01/dojotoolkit/dojox/image/tests/test_Magnifier.html)

[Adam Peller]

Replying to enzo_4:

But the file dojox/image/tests/images/spanke.jpg should be added to the changeset also, no? (it exists for ​http://archive.dojotoolkit.org/dojo-2011-02-01/dojotoolkit/dojox/image/tests/test_Magnifier.html)

Probably. I didn't fix the broken images. The ticket remains open.

[Enzo]

Ok for the missing image, not a problem indeed as other images are present in the test_Magnifier.html that let users test the magnifier

However, on ​http://archive.dojotoolkit.org/dojo-2011-09-09/dojotoolkit/dojox/image/tests/test_Magnifier.html, I can see your changes from changeset 26496, but the magnifier still doesn't work

To be more precise, when I move the mouse over an image, a magnifier is shown but much below in the page (not on the mouse), and it doesn't refresh when I move the mouse

I guess some coords problems to be involved in the bug

[Adam Peller]

Interesting. The problem you describe appears on FF, only with a build.

Well, the bug remains open.

[Enzo]

Yes, this is exactly my config for this test.

Some remarks that may help, or not, dunno...:

When I check ​http://archive.dojotoolkit.org/dojo-2011-09-09/dojotoolkit/dojox/image/Magnifier.js, compared to your ​http://bugs.dojotoolkit.org/browser/dojo/dojox/trunk/image/Magnifier.js?rev=26496

the built .js begins with: define("dojox/image/Magnifier",["dojo/_base/declare",...

and the non built Magnifier.js begins with: define(["dojo/_base/declare",...

=> So here the build process adds a first argument to define(), dunno if it's correct or not...

Another idea is that the build process change the name of the argument of: function(declare, construct, window, gfx, canvas, MagnifierLite) with: function(_1,_2,_3,_4,_5,_6)

but notice window argument that is a reserved name, maybe it can conflict with global window variable on some browsers like FF (and no conflict anymore with the built system that replaces window with _3)

Well really don't know for this problem.. just a couple of ideas, cause I deeply need the Magnifier on my FF :)

[dylan]

Given that no one has shown interest in creating a patch in the past 4+ years, I'm closing this as patchwelcome.