Skip to content

Conversation

@lianghx-319
Copy link
Member

Why

With iOS, it will apply developer nest this component in a form tag to get 「search」return button in keyboard. This enhancement lets us implement it call the component without extra next HTML element.

How

Show in the pr code review.

Test

Before

  • run yarn test
    test-before

  • runtime case
    ezgif com-video-to-gif

After

  • run yarn test
    test-after

  • runtime case
    ezgif com-video-to-gif (1)

@auto-add-label auto-add-label bot added the enhancement New feature or request label Oct 25, 2019
@netlify
Copy link

netlify bot commented Oct 25, 2019

Deploy preview for femessage-vant ready!

Built with commit 04294c2

https://deploy-preview-5--femessage-vant.netlify.com

@lianghx-319 lianghx-319 force-pushed the feat/enhance-search-input branch from c902112 to e003279 Compare October 25, 2019 04:56
);
}

if (props.removeForm) {
Copy link
Member

Choose a reason for hiding this comment

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

不需要 withForm
甚至可以不需要配置,直接套上 form

@levy9527 levy9527 merged commit 839a5c6 into FEMessage:femessage Oct 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants