Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent duplicate ids #48

Merged
merged 4 commits into from
Mar 26, 2017
Merged

Conversation

christiaan
Copy link
Contributor

When having a bigger amount of color pickers on a page it would happen that
multiple pickers had the same id.

By using a counter that increments for each color picker the id is now
guaranteed to be unique.

Also includes some missing var declarations.

Christiaan Baartse added 4 commits May 6, 2015 14:51
Because of this error the pageX variable is leaked onto the window object.
Hex is already declared as a function argument and thus does not need var.
When having a bigger amount of color pickers on a page it would happen that
multiple pickers had the same id.

By using a counter that increments for each color picker the id is now
guaranteed to be unique.
mrgrain added a commit to mrgrain/colpick that referenced this pull request May 27, 2015
@anoek
Copy link

anoek commented Jul 20, 2015

👍

@KaiserVito
Copy link

perfect

@josedvq josedvq merged commit 9911e07 into josedvq:master Mar 26, 2017
@christiaan christiaan deleted the bugfix_duplicate_ids branch March 27, 2017 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants