Skip to content

Conversation

@xhayper
Copy link
Contributor

@xhayper xhayper commented Dec 9, 2022

This PR make it so the component properly give out correct typing, and also extends HTMLAttribute

@xhayper xhayper marked this pull request as draft December 9, 2022 06:58
@xhayper xhayper marked this pull request as ready for review December 9, 2022 07:40
@cloydlau
Copy link
Owner

cloydlau commented Dec 9, 2022

Roger, a little busy at the moment, will get back to you ASAP.

@cloydlau cloydlau changed the title fix: typing type: give out types for boolean attributes Dec 11, 2022
@cloydlau
Copy link
Owner

It's great, thanks! Sorry for the delay.

I did some small changes:

  • I'd like to remain the naming for boolAttrs because in Vue the abbreviated version attrs is the conventional naming.
  • The default value should remain undefined because the boolean attributes could be true by default.

@cloydlau cloydlau merged commit a21140a into cloydlau:main Dec 11, 2022
@xhayper xhayper deleted the patch-1 branch December 11, 2022 08:30
@xhayper
Copy link
Contributor Author

xhayper commented Dec 11, 2022

It's fine for the delay! and i see! thank you for the information!

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

Successfully merging this pull request may close these issues.

2 participants