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

[mobile] Send button is too subtle #535

Closed
patestevao opened this issue Apr 4, 2018 · 3 comments
Closed

[mobile] Send button is too subtle #535

patestevao opened this issue Apr 4, 2018 · 3 comments

Comments

@patestevao
Copy link

Hi, I'm using Eclair 0.2.5 in Android 5.1.

Here is something I noticed.
The send button is one of the most important in this interface and it doesn’t call enough attention to itself.

Maybe the button could be in a brighter background color and/or slightly larger.
Let me know if I can help with some ideas and sketches on this.
Thanks!

For now, here is a screenshot with the button on the screen:
eclair_18

@dpad85
Copy link
Member

dpad85 commented Apr 4, 2018

Hi, this color is a bit muted because the primary color for the testnet version is gray-ish. The primary color for the mainnet version is much more vibrant (electric blue) and the button will be more visible. For the shape and position, we tried to respect the material guidelines, but honestly I don't have a strong opinion on this.

If you have any suggestions or sketches please don't hesitate. The github repository for eclair on mobile will be public very soon and contributions are welcome!

@patestevao
Copy link
Author

Hi, thanks for answering.

Nice to know that the color will be different in the mainnet version. Even so, you guys could consider making the button a different color from the main blue (like the "add" button in the Lightning channels section), it would get more attention that way.

I believe the positioning it's okay, users are used now to the material design standards. And the size might have been just my impression due to the discrete color.

Thanks for letting me know, I'll take a look at the repository when it's available.

@dpad85
Copy link
Member

dpad85 commented Apr 12, 2018

Moved this issue to eclair-wallet repo, please follow up in ACINQ/eclair-mobile#33

Thanks!

@dpad85 dpad85 closed this as completed Apr 12, 2018
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

No branches or pull requests

2 participants