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

Apply indicator to distinguish user's trades in order books. The indi… #362

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

peculiarity
Copy link

…cator is the cancel button. You can cancel offer from there as well not only from my orders widget.

image

Fixing #321

@gbalabasquer @pimato

@pimato
Copy link

pimato commented Nov 20, 2017

nice,

a few refinements:

bildschirmfoto 2017-11-20 um 22 26 25

  • could you give the cancel button column more width, and align it vertical and horizontal like you can see in the picture.
  • new svg´s for button in idle and hover mode:
    Archiv.zip

and after looking a few minutes on the code, could you clean up the code itself.I think there is a better way of implementing it.

…cator is the cancel button. You can cancel offer from there as well not only from my orders widget.
@peculiarity peculiarity force-pushed the feature/differenciate-user-orders-in-order-book branch from 2946810 to 1878153 Compare November 21, 2017 09:56
@peculiarity
Copy link
Author

@pimato Please verify the new look and code changes.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants