Skip to content

Conversation

paulatulis
Copy link
Contributor

@paulatulis paulatulis commented Dec 30, 2021

Attempt to address #9513

Screen Shot 2021-12-30 at 9 22 35 AM

2021-12-30 09 22 20

Because the combiner is a level deep, it shows up in each choice. Is that too busy for someone's eyes?

/>
))
.add('anyOf-array-schema', () => (
.add('oneOf-array-schema', () => (
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this change is unrelated to this issue, but I also updated the story names to match the files/schema

/>
))
.add('anyOf-array-schema2', () => (
.add('oneOf-array-schema2', () => (
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this change is unrelated to this issue, but I also updated the story names to match the files/schema

@paulatulis paulatulis marked this pull request as ready for review December 30, 2021 15:29
@paulatulis paulatulis requested review from a team and billiegoose December 30, 2021 15:29
Copy link
Contributor

@Nezteb Nezteb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried this out locally and it looks good. 👌

@mpodlasin
Copy link
Contributor

Yeah, I would be curious to hear from Nauman what would he expect in the select options 😅

If there is no array, you see only options in select, without anyOf/oneOf:

Screenshot 2022-01-04 at 15 12 18

Screenshot 2022-01-04 at 15 12 23

@mnaumanali94
Copy link
Contributor

If there is no array, you see only options in select, without anyOf/oneOf:

@mpodlasin I thought we already covered the case for anyOf/OneOf showing for non array cases.

@paulatulis paulatulis self-assigned this Jan 5, 2022
@paulatulis paulatulis merged commit 1f7bc29 into master Jan 5, 2022
@paulatulis paulatulis deleted the feat/show-child-combiners branch January 5, 2022 16:08
@stoplight-bot
Copy link
Collaborator

🎉 This PR is included in version 4.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants