Skip to content
New issue

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

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

Already on GitHub? # to your account

picker control cross-referencing #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

picker control cross-referencing #16

wants to merge 1 commit into from

Conversation

ackspony
Copy link

@ackspony ackspony commented Apr 6, 2017

The purposes of these changes are to facilitate different ways of manipulating / customizing the color picker control by CSS.

  1. Set the class name 'jscolor-picker-control' in the picker control wrap so it can be targeted in CSS.

  2. Generate a handle for picker instance and save it in the 'data-picker-handle' attribute in both the picker control wrap and callee for cross-reference.

  3. Save the input element's "id" attribute in the 'data-picker-input-id' attribute, (should it have the "id" attribute) for cross-reference.

  4. Save the input element's "class" attribute in the 'data-picker-input-class' attribute, (should it have the "class" attribute) for cross-reference.

The purposes of these changes are to facilitate different ways of manipulating / customizing the color picker control by CSS.

1. Set the class name 'jscolor-picker-control' in the picker control wrap so it can be targeted in CSS. 

2. Generate a handle for picker instance and save it in the 'data-picker-handle' attribute in both the picker control wrap and callee for cross-reference.

3. Save the input element's "id" attribute in the 'data-picker-input-id' attribute, (should it have the "id" attribute) for cross-reference. 

4. Save the input element's "class" attribute in the 'data-picker-input-class' attribute, (should it have the "class" attribute) for cross-reference.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant