search

poanetwork / token-wizard

(Discontinued) TokenWizard is an DApp to create and manage crowdsale and token contracts using a simple UI
MIT License
381 stars 216 forks source link

(Feature) Apply new design to step 5: "Crowdsale page" #1206

Closed gabitoesmiapodo closed 5 years ago

gabitoesmiapodo commented 5 years ago

Closes #1089

Description

Implementation of new styles for Step 5: "Crowdsale page"

Some screenshots:

screen shot 2018-10-31 at 15 12 58 screen shot 2018-10-31 at 15 13 09 screen shot 2018-10-31 at 15 13 18 screen shot 2018-10-31 at 15 13 27

screen shot 2018-10-31 at 15 06 05 screen shot 2018-10-31 at 15 06 21 screen shot 2018-10-31 at 15 06 32 screen shot 2018-10-31 at 15 06 40

BlackDuckCoPilot commented 5 years ago

Black Duck Security Report

Merging #1206 into integration/#1082-new-design will not change security risk.

Click here to see full report

dennis00010011b commented 5 years ago

@gabitoesmiapodo App failed to compile on MacOS I think it is problem with letter case sensitivity for paths

screen shot 2018-10-31 at 12 38 09 pm screen shot 2018-10-31 at 12 38 55 pm
gabitoesmiapodo commented 5 years ago

@dennis00010011b before I proceed with this, can you have a look at this? It's a fix for Git case sensitivity on MacOs (I had to use it not long ago): https://coderwall.com/p/5w3jca/git-case-sensitivity-on-os-x

I'm currently running this branch on MacOS and file paths and names seem to be OK.

screen shot 2018-10-31 at 17 04 23
dennis00010011b commented 5 years ago

@dennis00010011b before I proceed with this, can you have a look at this? It's a fix for Git case sensitivity on MacOs (I had to use it not long ago): https://coderwall.com/p/5w3jca/git-case-sensitivity-on-os-x

@gabitoesmiapodo command git config core.ignorecase true doesn't work for me in this case. Build is succefully if directory renamed from 'src/components/crowdsale' to 'src/components/Crowdsale'

dennis00010011b commented 5 years ago

@gabitoesmiapodo Would be usable to add icon copy for Crowdsale Proxy Address field

screen shot 2018-10-31 at 2 25 06 pm

gabitoesmiapodo commented 5 years ago

@dennis00010011b

- Would be usable to add icon copy for Crowdsale Proxy Address field

Sure, done.

dennis00010011b commented 5 years ago

@gabitoesmiapodo Publish page formatted in wrong way

screen shot 2018-11-01 at 4 11 46 pm
gabitoesmiapodo commented 5 years ago

@dennis00010011b

That was changed in Step 4's branch. Brought the changes to this branch anyway, it should look fine now.