Skip to content
This repository has been archived by the owner on Aug 25, 2022. It is now read-only.

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

abrenoch
Copy link

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.

…nova#54)

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

beck24 commented Dec 14, 2014

Tested - solution works well

@kim3er
Copy link

kim3er commented Nov 18, 2015

Would like to see this merged.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants