search

websanova / wPaint

jQuery Paint Plugin
http://wpaint.websanova.com
351 stars 128 forks source link

Proposed solution to the path variable being incremented (Issue #54) #87

Open abrenoch opened 10 years ago

abrenoch commented 10 years ago

I noticed there are a couple proposed solution to this issue, however I believe this would be the most effective method.. The root of the problem appeared to be that the original object array was being modified, and every time the plugin was reinitialized that array was being modified over again.

Both of the other solutions basically just check the img variable do determine whether it exists (#73) or has already had the path variable prepended to it (#81).

This should create a duplicate of the $.fn.wPaint.menus[this.name] object rather than referencing and modifying the original.

beck24 commented 9 years ago

Tested - solution works well

kim3er commented 8 years ago

Would like to see this merged.