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

(Feature) Apply new design to "No Web3 Available" section #1217

Conversation

gabitoesmiapodo
Copy link

Closes #1178

Description

Updated visuals for this feature / section.

Some screenshots / what to expect

desktop
tablet
mobile xl
mobile sm

…new-design-to-noweb3-available

* integration/#1082-new-design:
  Fix wording
  Remove duplicate error on whitelist
  Added test to copy
  Update validations tests
  Add file upload improvements for step3 and tests
…new-design-to-noweb3-available

* integration/#1082-new-design:
  Fix typo
  Fix tests
  Fix integration branch
  Prettier
  Apply reload on crowdsales list
  Update snapshots
  Improvement to check metamask extension on home and stepOne
  Reload from home to stepOne
@dennis00010011b
Copy link

@gabitoesmiapodo
Warning is briefly displayed even if wallet installed and logged in
https://www.useloom.com/share/744994a83e6143789791a2b12f855324

@gabitoesmiapodo
Copy link
Author

@dennis00010011b

IIRC that was already happening.

This is just for visuals.

Copy link
Contributor

@fernandomg fernandomg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, kudos for the tests! 👏

@mariano-aguero
Copy link
Contributor

LGTM, @gabitoesmiapodo will create an issue to solve the visual problem when the status is changed

@mariano-aguero mariano-aguero merged commit 33f2b6a into integration/#1082-new-design Nov 8, 2018
@ghost ghost removed the in progress label Nov 8, 2018
@mariano-aguero mariano-aguero deleted the feature/#1178-apply-new-design-to-noweb3-available branch November 8, 2018 12:12
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

Successfully merging this pull request may close these issues.

None yet

5 participants