This repository has been archived by the owner on Oct 11, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 17
design review #51
Comments
Hey David, here are my suggestions:
|
@mebates now that autocomplete is being added here is how things look when the search icon is moved to the left. Notice how I added some left padding to align the autocomplete results with the search box text. Do you think that's good? Or should we just leave the padding alone like this: |
Merged
Hey @himedlooff : I prefer your first attempt, with the extra padding on the left to align under what you're typing. |
Can we close this, or remove me from assigned? =D |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Several things wanted to check with you on:
Will probably be back with a couple more once we have autocomplete implemented.
The text was updated successfully, but these errors were encountered: