-
Notifications
You must be signed in to change notification settings - Fork 4
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
Do we need EyeDropper instances? #18
Comments
I've always found that |
I kind of assumed there was going to be some statefulness to the If there's no state and this is purely a function, then |
(FWIW, |
@ipopescu93 What are your current thoughts? As we're thinking about adding
|
It is expected that state is going to be added for future capabilities of the |
Why can't that be an options dictionary to the hypothetical |
it is already shipped, does any one think this will change any more? |
It shipped in one browser only, I'm somewhat hopeful this can still be changed. |
Also see #35 |
It seems this could just be a method on navigator or some such. Or perhaps a static method on EyeDropper if we wanted to namespace it. @domenic probably knows what's modern.
The text was updated successfully, but these errors were encountered: