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

Request: on-hover change color? #47

Closed
TheSayantan96 opened this issue Dec 29, 2018 · 3 comments
Closed

Request: on-hover change color? #47

TheSayantan96 opened this issue Dec 29, 2018 · 3 comments
Assignees

Comments

@TheSayantan96
Copy link

Sir, is there a way we can change the color on mouse hover over the color picker?

@jaames jaames self-assigned this Dec 29, 2018
@jaames
Copy link
Owner

jaames commented Dec 29, 2018

Not really; you can already click and then drag to change the color, so I don't see why changing the color on mouse hover is necessary.

As I see it, it wouldn't be very intuitive for the user, and it wouldn't work on touchscreen devices anyway.

What kind of use-case do you have for this?

@TheSayantan96
Copy link
Author

TheSayantan96 commented Dec 29, 2018

Sir that I know, but what I want is that when I am moving the mouse cursor inside the color picker the respective color should show instantly. Here's the link of what I am trying to do http:https://pal-lightings.herokuapp.com/

So, what I want is that, when I am moving the cursor in side the color picker without any click event the respective color should show beside in the div. Sir, it would be great if you can help me with that.

@jaames
Copy link
Owner

jaames commented Dec 29, 2018

I understand, but unfortunately this isn't possible with iro.js.

The problem with changing the color on mouse hover is that if the user starts hovering to select a color and then moves the mouse over to something else on the page, the hover events will still fire and the color picker will be left on a color that the user didn't intentionally choose.

Plus, as mentioned, there are no hover events on touchscreen devices like phones and tablets. One of the core features of iro.js is that it supports multiple devices and I don't really want to add an option that would break this feature.

Sorry to disappoint, good luck with your project though! :)

@jaames jaames closed this as completed Dec 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants