-
Notifications
You must be signed in to change notification settings - Fork 578
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
Template delete button/kira pilot #4992
Conversation
<Typography variant="subtitle2"> | ||
{Language.ttlHelperText(form.values.default_ttl_ms)} | ||
</Typography> | ||
<span>{Language.ttlHelperText(form.values.default_ttl_ms)}</span> |
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.
Changed this because I thought it was too big. Now it matches the other helper text in the form.
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.
@jsjoeio thanks for the review!
Just want to make sure I understand: we do show the name of the template as the user begins typing, but you're saying we should show it in the modal description to be clearer. Is that right? |
Exactly! |
resolves #4980