-
Notifications
You must be signed in to change notification settings - Fork 12
feat(Settings.Item): add the ability to hide item title #492
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
Conversation
|
Preview is ready. |
|
Playwright Test Component is ready. |
| @@ -1,7 +1,6 @@ | |||
| @use '~@gravity-ui/uikit/styles/mixins'; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For what?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To reduce amount of warnings about @import deprecation
src/components/Settings/Settings.tsx
Outdated
| renderTitleComponent = identity, | ||
| mode, | ||
| description, | ||
| showTitle = true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we use the result of renderTitleComponent instead of creating a new prop?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mean when we return null from it?
c85d980 to
600f4b7
Compare
600f4b7 to
cb659ef
Compare
No description provided.