search

bitshares / bitshares-community-ui

[Worker Proposal] Light and 100% Responsive, BitShares Community DEX/UI - based on Vue.js Framework
MIT License
16 stars 10 forks source link

392 several bugs #395

Closed lapinlapin closed 5 years ago

lapinlapin commented 5 years ago

What's this PR do?

Made: 392,382,379,363,362,354,340

Where should the reviewer start?

How should this be manually tested?

https://localhost:8080/

Any background context you want to provide?

What are the relevant tickets?

392

382

379

363

362

354

340

Screenshots (if appropriate)

URL

Provide a link to your deploy from http://staging-ui.trusty.apasia.tech:8080/branch-392-several-bugs

Questions:

roma219 commented 5 years ago

@lapinlapin check again #392 Remember to go through all the issues and double-check before submitting a PR.

roma219 commented 5 years ago

@lapinlapin What is not understandable about that:

@lapinlapin check again #392 Remember to go through all the issues and double-check before submitting a PR.

lapinlapin commented 5 years ago

Check it.

  1. Revert Confirm Order modal to be wider & display values in column on mobile - FIXED
  2. New Order -> Select Market tab -> select Sell - FIXED
  3. Errors - FIXED
  4. this screen does not exist already
  5. Double border - FIXED
  6. Confirm order modal reversed assets - FIXED
  7. Get and Spend Inputs are too far away, make it a bit closer - FIXED
roma219 commented 5 years ago

@lapinlapin