-
Notifications
You must be signed in to change notification settings - Fork 65
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
New EyeDropper feature #1179
New EyeDropper feature #1179
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure if I should suggest eye-dropped
instead of eyedropper
. The spec strangely says EyeDropper
instead of Eyedropper
. But https://helpx.adobe.com/photoshop/using/tool-techniques/eyedropper-tool.html suggests that web developers might be more familiar with "Eyedropper" than "Eye dropper".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @captainbrosset! The test failure here is due to #1182, so this can now merge failing tests notwithstanding cannot merge due to our branch protection rules. The other PR needs to land first. 🤦
No description provided.