Augmint / augmint-web

Augmint Web Frontend
http://www.augmint.org
GNU Affero General Public License v3.0
17 stars 8 forks source link

Improve top nav and account overview layout #630

Closed admrid closed 5 years ago

admrid commented 5 years ago

Nature of the PR: bug

Steps to reproduce:

Trello card / screenshot / wireframe link:

Is connection necessary to test? If so which network?

szerintedmi commented 5 years ago

Deploy preview for augmint ready!

Built with commit 96461ac6d68a1a209dbb453b3b851bc8e803e90f

https://deploy-preview-630--augmint.netlify.com

admrid commented 5 years ago
  • [ ] too many decimal zeros in ETH balance

  • [ ] horizontal scroll in my account history on 320px

  • [ ] My total loans and total locks is breaking into two lines too early

these are not really about the accountInfo comp. and i am anyway refreshing the /account page so let's push them to that PR. (there is tho one small fix coming related to the accInfo comp)

szerintedmi commented 5 years ago

these are not really about the accountInfo comp. and i am anyway refreshing the /account page so let's push them to that PR. (there is tho one small fix coming related to the accInfo comp)

Right, moved to https://github.com/Augmint/augmint-web/issues/657

admrid commented 5 years ago

@phraktle i think this can be merged now. all the other points of yours are going to be addressed in another PR

phraktle commented 5 years ago

@admrid is the design change making the upper right panel white instead of the previous yellow intentional or a bug?

admrid commented 5 years ago

@phraktle intentional: i think for now it is good to have those pieces of information presented there, but those should not take the focus off from the main content/actions of the app, hence there is no need for those to be visually this strong.