(Improvement) UI Enhancement and suggestions

poanetwork / poa-popa

DApp for proof of physical address (PoPA) attestation for validators of POA Network

https://popa.poa.network

GNU General Public License v3.0

24 stars 18 forks source link

(Improvement) UI Enhancement and suggestions #175

Closed SiddigZeidan closed 6 years ago

SiddigZeidan commented 6 years ago

Logo looks blurry. Needs a retina version
The box-shadow on the hover button is too thick. I'd recommend changing to box-shadow: inset 0 0 0 1px #5c34a2; while look like the screenshot below
Bold the how-to-title titles on
Change 0.04 ETH to 0.04 POA
When I click "Continue" there is no refresh and the URL changes to the next page in a smooth manner. This is not the case when I click "Back" as it seems to refresh the page
Country should be disabled or read-only
All US armed forces should be at the bottom and should be together

igorbarinov commented 6 years ago

Change to 0.04 POA on POA

It's a good question about a different price for diff networks @pablofullana do we have an issue for that?

pablofullana commented 6 years ago

No we don't. This would be required for PoPA as well, correct?

igorbarinov commented 6 years ago

I think it should be in config for each chain on all proof of apps

phahulin commented 6 years ago

We have priceWei option in server-config, so we need to "forward" it to UI.

I'd like to point out that in the current implementation server can only support working with a single network, so we'll need to have multiple instances, one per chain, if that is required. The currently deployed version at popa....network works with Core.

igorbarinov commented 6 years ago

Sounds good. Let's point UI to config and specify 1 POA for verification