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

Share docs between compound component of the same parent #3434

Merged
merged 7 commits into from
Dec 4, 2024

Conversation

ethanshar
Copy link
Collaborator

Description

In order to share docs between compound components of the same parent I copy the docs entry from the parent to children
for example components like (TabController and its children TabController.TabBar, TabController.TabPage, etc...)

Changelog

Share docs between compound components

Additional info

@Inbal-Tish
Copy link
Collaborator

@ethanshar I'm not sure I understand this PR... How will the TabController docs will look like if we want to add docs to both TabBar (exists) and TabPage components?

@ethanshar
Copy link
Collaborator Author

@ethanshar I'm not sure I understand this PR... How will the TabController docs will look like if we want to add docs to both TabBar (exists) and TabPage components?

I discussed with Yulia, she also preferred to remove 🙄 so I removed

@ethanshar ethanshar requested a review from Inbal-Tish December 4, 2024 11:55
@ethanshar ethanshar assigned Inbal-Tish and unassigned ethanshar Dec 4, 2024
@Inbal-Tish
Copy link
Collaborator

@ethanshar I'm not sure I understand this PR... How will the TabController docs will look like if we want to add docs to both TabBar (exists) and TabPage components?

I discussed with Yulia, she also preferred to remove 🙄 so I removed

Then why you did remove the docs for the TabBar after moving them to the TabController? Now there is a duplication...

@Inbal-Tish Inbal-Tish assigned ethanshar and unassigned Inbal-Tish Dec 4, 2024
@Inbal-Tish Inbal-Tish enabled auto-merge (squash) December 4, 2024 13:46
@Inbal-Tish Inbal-Tish merged commit b29302e into master Dec 4, 2024
1 check passed
@Inbal-Tish Inbal-Tish deleted the infra/compound_components_docs branch December 4, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants