h.toLowerCase, error in js library (jpicker.min.js)

[GoogleCodeExporter]

What steps will reproduce the problem?
1. draw curve with a pencil
2. change fill color

What is the expected output? What do you see instead?
get error in js-code. See the attached files

In what browser did you experience this problem?
ALL

In what version of SVG-edit does the problem occur? 
Latest trunk

Error in this part of code:
validateHex : function(h) {
    console.log(h);
    h = h.toLowerCase().replace(/[^a-f0-9]/g, "");
    if (h.length > 8) {
        h = h.substring(0, 8)
    }
    return h
},

Original issue reported on code.google.com by sebastia...@gmail.com on 3 Dec 2011 at 2:18

Attachments:

• 1.png

[GoogleCodeExporter]

Error text: h.toLowerCase is not a function

Original comment by sebastia...@gmail.com on 4 Dec 2011 at 11:42

[GoogleCodeExporter]

In my project such error occurs then fill attribute is set to none. Changing to 
some color(#000 f.e.) is solving the problem.

Original comment by divine.i...@gmail.com on 17 Aug 2012 at 1:00

[GoogleCodeExporter]

Yes, probably related to issue 935. Basically the root is jQuery update causing 
errors in 3rd-party library like jPicker.

Original comment by asyazwan on 17 Aug 2012 at 3:14

• Changed state: Needs3rdPartyFix

[GoogleCodeExporter]

Any progress on this issue?

Original comment by flip.sch...@gmail.com on 19 Nov 2012 at 9:50

[GoogleCodeExporter]

Hi,

No fix yet AFAIK. We'll probably have to patch this our own before 2.6 release 
and submit the patch to the project: https://code.google.com/p/jgraduate/.

Changing the status of this ticket as it seems like a big deal to me.

Original comment by asyazwan on 20 Nov 2012 at 1:28

• Changed state: Accepted
• Added labels: NeededFor-2.6, Priority-Critical
• Removed labels: Priority-Medium

[GoogleCodeExporter]

Changing the owner to Mark MacKay.

Original comment by asyazwan on 4 Dec 2012 at 6:21

[GoogleCodeExporter]

Ok, this needs confirmation, but it's tentatively fixed in r2225

Original comment by duopi...@gmail.com on 5 Dec 2012 at 12:08

• Changed state: Fixed