Skip to content

Conversation

@emplums
Copy link

@emplums emplums commented Sep 30, 2019

@rafeca in #559 pointed out some inconsistencies with our Details props + typings - updating here! Will also update the release notes for 14.0.0

@vercel
Copy link

vercel bot commented Sep 30, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://primer-components-git-details-docs.primer.now.sh

@emplums emplums requested a review from colebemis September 30, 2019 16:34
@vercel vercel bot temporarily deployed to staging September 30, 2019 16:34 Inactive
@vercel vercel bot temporarily deployed to staging September 30, 2019 16:35 Inactive
@emplums emplums merged commit 720ce85 into master Sep 30, 2019
@emplums emplums deleted the details-docs branch September 30, 2019 16:38
@dmarcey
Copy link
Contributor

dmarcey commented Sep 30, 2019

Do we want to omit the open prop in the TypeScript declare file? I think it looks like it's there to TypeScript consumers because I extend the native attributes for the <details /> element, which has open I think.

https://github.com/primer/components/pull/544/files#diff-b52768974e6bc0faccb7d4b75b162c99R101

@rafeca
Copy link
Contributor

rafeca commented Sep 30, 2019

Thanks for the fix! You should be able to enable this disabled test by also changing the open property to defaultOpen.

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.

4 participants