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

Incorrect width by default #137

Open
fjeddy opened this issue Mar 12, 2022 · 4 comments
Open

Incorrect width by default #137

fjeddy opened this issue Mar 12, 2022 · 4 comments

Comments

@fjeddy
Copy link

fjeddy commented Mar 12, 2022

image

No changes, simply use the component. It seems to be including the padding as part of the inner content width, and therefor it doesn't look proper?

The width prop is not working either, even tho a PR exists to add it, it's not actually present. So maybe it doesn't exist anymore?

@fjeddy
Copy link
Author

fjeddy commented Mar 12, 2022

It's caused by the lack of a box-sizing: content-box; so it's inheriting whatever the parent is using.

@gmasil
Copy link

gmasil commented Apr 9, 2022

Thanks for pointing this out! I hope this will be added by default to the component...

@olojiang
Copy link

Suffer same issue now, it's should be fixed by change the following part
totalWidth(): number {
return this.hueHeight + (this.hueWidth + 8) * 2
},

it makes no sense, if above part is overflow the box, but below part is ok

@runger1101001
Copy link

Same problem here, in Safari...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants