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

ColorPicker components: Input fields style issue #12647

Closed
NenadObradovic opened this issue Dec 6, 2018 · 0 comments · Fixed by #12747
Closed

ColorPicker components: Input fields style issue #12647

NenadObradovic opened this issue Dec 6, 2018 · 0 comments · Fixed by #12747
Labels
Good First Issue An issue that's suitable for someone looking to contribute for the first time

Comments

@NenadObradovic
Copy link

NenadObradovic commented Dec 6, 2018

I found the issue with visibility of color picker value when you change the color format. I just add simple colorpicker field inside InspectorControls https://www.screencast.com/t/F6GrEHN7EQ

el(ColorPicker, {
	type: 'color',
	label: i18n.__('Title Color'),
	value: attributes.titleColor,
	onChangeComplete: function (newTitleColor) {
		props.setAttributes({ titleColor: newTitleColor })
	}
}),

and when you click to change the color format, other values are not visible https://www.screencast.com/t/MVLiGgy3XK8o

Problem is default padding value for input fields padding: 6px 8px;. When you reduce these values or increase input fields size then you will see color value https://www.screencast.com/t/uQLqAb0V1n

WordPress: 5.0-RC3-43968
Google Chrome: 71.0.3578.80

Best regards,
Nenad

@jorgefilipecosta jorgefilipecosta added the Good First Issue An issue that's suitable for someone looking to contribute for the first time label Dec 7, 2018
@IreneStr IreneStr assigned IreneStr and unassigned IreneStr Dec 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue An issue that's suitable for someone looking to contribute for the first time
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants