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

feature(helpmessage): change type helpmessage #940

Merged
merged 4 commits into from
Dec 10, 2021

Conversation

samuel-gomez-axa
Copy link
Contributor

Related issue

#939

Description of the issue

image

image

Person(s) for reviewing proposed changes

@arnaudforaison

@samuel-gomez-axa samuel-gomez-axa added the Type: Enhancement New feature or request label Dec 3, 2021
@samuel-gomez-axa samuel-gomez-axa linked an issue Dec 3, 2021 that may be closed by this pull request
@samuel-gomez-axa samuel-gomez-axa self-assigned this Dec 3, 2021
@arnaudforaison arnaudforaison force-pushed the feature/allow-jsx-help-message-#939 branch 2 times, most recently from 481c0ad to 12321d9 Compare December 10, 2021 10:44
@samuel-gomez-axa samuel-gomez-axa force-pushed the feature/allow-jsx-help-message-#939 branch from 12321d9 to 11992bc Compare December 10, 2021 14:55

type HelpProps = {
message?: string;
type HelpProps = HTMLProps<HTMLSpanElement> & {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
type HelpProps = HTMLProps<HTMLSpanElement> & {
type HelpProps = React.ComponentPropsWithoutRef<"small"> & {

@samuel-gomez-axa samuel-gomez-axa merged commit 4665f4f into master Dec 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow JSX in Helpmessage
2 participants