Skip to content
This repository has been archived by the owner on Nov 10, 2023. It is now read-only.

Ignore trailing/leading spaces on input fields #1027

Closed
lukasschor opened this issue Jun 17, 2020 · 1 comment · Fixed by #1142
Closed

Ignore trailing/leading spaces on input fields #1027

lukasschor opened this issue Jun 17, 2020 · 1 comment · Fixed by #1142
Assignees
@matextrem @dasanra
Labels
Bug 🐛 Something isn't working Major Needs to be fixed for immediate next public release.
Milestone
Sprint 9

Comments

@lukasschor
Copy link
Member

lukasschor commented Jun 17, 2020 •
edited
Loading

When loading a Safe and copy-pasting and address, sometimes a trailing/leading space can be copied over. This results in an error and leaves the user confused as it is a valid Safe address. We should ignore those spaces for address input fields (and maybe also other input fields?).

image.png
@lukasschor lukasschor added Bug 🐛 Something isn't working Major Needs to be fixed for immediate next public release. labels Jun 17, 2020
@tschubotz
Copy link
Member

I think also leading spaces should be ignored

@lukasschor lukasschor changed the title Ignore trailing spaces on input fields Ignore trailing/leading spaces on input fields Jun 17, 2020
@matextrem matextrem self-assigned this Jul 16, 2020
@matextrem matextrem added this to the Sprint 9 milestone Jul 17, 2020
@dasanra dasanra mentioned this issue Jul 22, 2020
Merged
@mmv08 mmv08 mentioned this issue Aug 7, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@matextrem matextrem

@dasanra dasanra

Labels
Bug 🐛 Something isn't working Major Needs to be fixed for immediate next public release.
Projects
None yet
Milestone
Sprint 9
Development

Successfully merging a pull request may close this issue.

Trim spaces from AddressInput 5afe/safe-react
4 participants
@tschubotz @matextrem @lukasschor @dasanra