Skip to content

Conversation

@faisalman
Copy link
Contributor

Hi, thanks for the types! I'm using it and just made a small update to also include the recently added formFactor data: https://wicg.github.io/ua-client-hints/#sec-ch-ua-form-factor

@lukewarlow
Copy link
Owner

Thanks for this!

@lukewarlow lukewarlow merged commit 04c39f1 into lukewarlow:master Sep 7, 2023
@lukewarlow
Copy link
Owner

This is released in 0.4.1. The type as per https://wicg.github.io/ua-client-hints/#dom-uadatavalues-formfactor is actually just a string not a string[] just in case you notice that difference when you upgrade.

Thanks for pointing out I was missing this property!

@faisalman
Copy link
Contributor Author

I was referring to it at first, but judging by this commit (that's just pushed yesterday): WICG/ua-client-hints@65215d8 I just thought it might eventually get changed into a string[] later.

@lukewarlow
Copy link
Owner

Ah thanks for pointing me to that. I'll keep an eye out for any changes and am happy to update the types once the Ua Data Spec / Chrome implementation changes :)

@lukewarlow
Copy link
Owner

I've left a comment WICG/ua-client-hints#343 (comment) trying to get clarification on this

@lukewarlow
Copy link
Owner

@faisalman as an update the spec has been corrected so I've changed it back to an array and released 0.4.2. Apologies for that back and forth. Thanks once again!

@faisalman
Copy link
Contributor Author

Thanks to you! Happy to help 👍

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