Closed jmanke closed 1 year ago
This would be a nice QoL improvement.
@benelan I triaged this one to this milestone since I already had something in the works for it.
Installed and assigned for verification.
Unfortunately, in beta.94
it doesn't let you paste valid hex at all now. Also, should it be using internalSetValue()
instead of only setting the Input's value?
The way it is now won't update Color Picker's value
property, as well as some other states like previousNonNullValue
and internalColor
. It also won't emit, which is taken care of in internalSetValue()
.
https://codepen.io/benesri/pen/vYjgQBX?editors=100
Edit:
Nevermind on the internalSetValue()
part, it looks like that should happen when Input's value changes.
Taking another look! 👀
This issue will be prioritized after the 1.0 release next month.
This was addressed via https://github.com/Esri/calcite-components/issues/749 and will be available in the next release (see updated demo). cc @geospatialem
Verified in 1.4.0-next.9
🎨
I tried this with 1.4.0 and I don't think it works correctly.
When you paste a new hex value the color does not change. It also does not change if you click anywhere else outside of the hex box afterwards.
If you type in a new hex value the color only changes if you click directly on the RGB/HSV label or the +/- buttons. Clicking just outside of the box doesn't apply the new color. I even think it should automatically apply the color as soon as a valid hex value got entered, not only on blur.
Thanks for catching this, @AdelheidF. Need to improve test coverage for this.
I'll take a look to see if this can be fixed for the upcoming patch.
Installed and assigned for verification.
3rd time's the charm, right? 😄🍀
When you paste a new hex value the color does not change. I even think it should automatically apply the color as soon as a valid hex value got entered, not only on blur.
I do want to clarify that there was no change regarding ☝️. Pasted values will get processed on blur or on Enter
press. cc @ashetland
Verified when on blur or the Enter
key is pressed per Franco's note above.
Description
Currently, pasting a color with the full hex string (ex. #FF00FF) into the hex input will not work. The last character is truncated and the value is not accepted. This is something users routinely do in Dashboards.
Acceptance Criteria
Pasting a full hex string (with the # symbol included) should update the color-picker's color.
Relevant Info
No response
Which Component
calcite-color-picker
Example Use Case
A user is configuring their theme using the calcite-color-picker and would like to use hex colors from an external source. The user will attempt to paste a hex color into the hex input but it will fail to update the color. A user would expect the hex input to accept hex strings that include the # symbol.