Skip to content

fix: remove "show" toggles on descendant component#4711

Merged
TrySound merged 2 commits intomainfrom
hide-desc
Jan 4, 2025
Merged

fix: remove "show" toggles on descendant component#4711
TrySound merged 2 commits intomainfrom
hide-desc

Conversation

@TrySound
Copy link
Contributor

@TrySound TrySound commented Jan 4, 2025

Closes #4670

@TrySound TrySound requested review from johnsicili and kof January 4, 2025 09:27
Copy link
Member

@kof kof left a comment

Choose a reason for hiding this comment

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

Another option would have been to add this ability to meta to have show toggle available, just like we do that with props

@TrySound
Copy link
Contributor Author

TrySound commented Jan 4, 2025

It's ok to have custom behavior for core components. I doubt this is the case for react components.

@TrySound TrySound merged commit 85147c8 into main Jan 4, 2025
7 checks passed
@TrySound TrySound deleted the hide-desc branch January 4, 2025 13:28
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.

Remove "show" prop and eye button from descendants

3 participants