search

Letractively / web-to-print-scripts

Automatically exported from code.google.com/p/web-to-print-scripts
0 stars 0 forks source link

Color picker BUG #34

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
The script for the color picker is broken. Color picker is not working on
the w2p site.

Try it here:
http://realestate.zetaprints.com/?page=template;TemplateID=B2F4B4CC-A128-440C-83
73-169E5F4FB50E

No matter where I click, it selects a color from the top edge of the picker
color area.

At the moment, realestate.zetaprints.com has this in the JS settings box:
http://realestate.zetaprints.com/java/test/1/jquery.min.js
http://zetaprints.com/java/dev/imageEditor.js
http://zetaprints.com/java/dev/multiFileUpload.js
http://zetaprints.com/java/dev/fileUpload.js
http://zetaprints.com/java/dev/colorPicker.js

I tried the color picker on http://zetaprints.com and it works ok, but that
system has no additional colorpicker.js script assigned. So, I'm guessing
this script is making it fall apart.

Original issue reported on code.google.com by agur...@gmail.com on 28 May 2010 at 12:53

GoogleCodeExporter commented 8 years ago

Original comment by trilobit.alex@gmail.com on 15 Jun 2010 at 10:02

GoogleCodeExporter commented 8 years ago 
bug has been fixed

Original comment by trilobit.alex@gmail.com on 15 Jun 2010 at 2:22

GoogleCodeExporter commented 8 years ago 
Color picker behavior is back to normal

Original comment by agur...@gmail.com on 15 Jun 2010 at 2:37

GoogleCodeExporter commented 8 years ago 
What was the fix?
Why did we have it in the first place?

Original comment by zetapri...@gmail.com on 15 Jun 2010 at 7:44

GoogleCodeExporter commented 8 years ago

Original comment by zetapri...@gmail.com on 16 Jun 2010 at 3:02

GoogleCodeExporter commented 8 years ago 
SVN release
http://code.google.com/p/web-to-print-scripts/source/detail?r=113

The URL was changed to the new
http://zetaprints.com/java/dev/201006/14/colorPicker.js

How it works.
The problem was in incorrect calculation Y-coord for crosshair. Now it uses 
crossplatform calculation for scroll of page.

Original comment by trilobit.alex@gmail.com on 16 Jun 2010 at 3:26

GoogleCodeExporter commented 8 years ago

Original comment by zetapri...@gmail.com on 28 Jun 2010 at 10:35