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

Enhance address input #3065

Merged
merged 2 commits into from
Nov 1, 2016
Merged

Enhance address input #3065

merged 2 commits into from
Nov 1, 2016

Conversation

ngotchac
Copy link
Contributor

@ngotchac ngotchac commented Nov 1, 2016

Closes #3057

Adds 0x to an address if it makes it valid. Display identity icon only on valid addresses.

@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. M6-ui labels Nov 1, 2016
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 12101f4 on ng-ehanced-address-input into * on master*.

derhuerst added a commit that referenced this pull request Nov 1, 2016
@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Nov 1, 2016
@jacogr jacogr merged commit bb120ec into master Nov 1, 2016
@ngotchac ngotchac deleted the ng-ehanced-address-input branch November 1, 2016 22:52
gavofyork pushed a commit that referenced this pull request Nov 3, 2016
* basic address autocomplete

* validate input, propagate changes

* show IdentityIcon in menu

* show IdentityIcon next to input

* refactoring, better variable names, linting

* show default IdentityIcon if search by name

* port #3065 over

* show accounts in the beginning

* show accounts before contacts

* filter deleted accounts

* UX improvements

- limit number of search results shown
- hint text

* only render identity icon if valid address

* UX improvements

- align IdentityIcon
- better hint text

* align label & error with other inputs

This probably needs to be changed soon again. Therefore this ugly hack has been put in place.

* Align component with coding style for app

* Use standard/tested AddressAutocmplete (WIP)

* Address selection & inputs operational

* Update TODOs, remove unused CSS

* only handle input changes when editing

* Simplify

* Cleanup unused modules

* Add contracts to address search

* Updates Address Selector to handle valid input address #3071

* Added Address Selector to contracts read queries
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants