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

feat: Add submit form on pressing enter, Closes #154 #227

Closed
wants to merge 1 commit into from

Conversation

kyranjamie
Copy link
Collaborator

@kyranjamie kyranjamie commented Feb 25, 2020

Didn't figure out how to do this with the keyCode, isn't in the event (even with .persist()). But checking input for line return char works well.

On pressing return, check for line return, if it's there, blur text area and submit form.

@hstove hstove self-requested a review February 25, 2020 15:36
@markmhendrickson markmhendrickson requested review from aulneau and removed request for hstove February 25, 2020 15:37
@hstove hstove self-requested a review February 25, 2020 15:37
@kyranjamie
Copy link
Collaborator Author

Can this be merged, though? Or should we wait for monorepo to be fully functional?

@markmhendrickson
Copy link
Collaborator

@hstove will make this change in the monorepo

@hstove
Copy link
Contributor

hstove commented Feb 26, 2020

Closing - changes ported in #235

@hstove hstove closed this Feb 26, 2020
@hstove hstove deleted the fire-on-return branch February 26, 2020 20:57
kyranjamie pushed a commit that referenced this pull request Aug 26, 2024
## [6.46.0](v6.45.2...v6.46.0) (2024-08-26)

### Features

* change balance load logic, closes leather-io/issues[#227](#227) ([faf71fb](faf71fb))

### Bug Fixes

* add default true to show settings to make sure it shows for locked screens, closes [#5777](#5777) ([96b2990](96b2990))
* do not capitalise add network ([5258d7e](5258d7e))
* empty space in settings dropdown ([d33df6f](d33df6f))
* failing ledger test ([67799fe](67799fe))
* fix formatting of add network ui, closes leather-io/issues[#5735](#5735) ([8a76b5f](8a76b5f))
* fix issues with alignment of page headers ([ea76352](ea76352))
* **ledger:** support stx_signMessage, closes [#5756](#5756) ([92a99ed](92a99ed))
* make headers more composable, prevent logo hover, closes leather-io/issues[#4597](#4597) ([3f2e359](3f2e359))
* minimum bitcoin spend, closes [#5739](#5739) ([34405f6](34405f6))
* prevent illegal constructor error ([a774b66](a774b66))
* remove blockstream from tests and add network form ([86af370](86af370))
* remove card from retrieve taproot ui ([27ca424](27ca424))
* **sentry:** sample performance units, closes leather-io/issues[#249](#249) ([3b6a5d8](3b6a5d8))
* tool to clear submitted tx state ([1e6f63d](1e6f63d))
* update ui library and deprecate footer ([0f5cfa2](0f5cfa2))
* validate dom nesting console error ([fe21fa8](fe21fa8))

### Internal

* differentiate action popup and popup, improve responsiveness and refactor footers, ref [#5260](#5260) ([2cae2a2](2cae2a2))
* hide lock option if wallet already locked ([b43cf8b](b43cf8b))
* post-release merge back ([686a823](686a823))
* remove back button on unlock screen and remove fragment ([9bc13ab](9bc13ab))
* update monorepo dependancies ([427f82f](427f82f))
* update prettier ([9e7d404](9e7d404))
* update tanstack libraries ([1486f5c](1486f5c))
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.

4 participants