-
Notifications
You must be signed in to change notification settings - Fork 569
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
[2][kapeer42] Exchange Vertical OrderBook Centering Issues #1689
Comments
Any progress on this @mattbeckman? |
I'd suggest that we remove the centering completely and let the user scroll determine where this is positioned. Thoughts? |
Sorry, forgot I self assigned. I started work on it, and I think we should split the bid/ask/center so that we aren't trying to center vertically across a large set of rows. Separate scroll for top/bottom. I'll get a screen capture soon. |
By adding the following to the code here we can handle centering much better
This makes this redundant in the prepending statement on line 362 |
@mattbeckman will you complete this issue? I think we should just set the order book market price in the middle and have two separate columns that can be scrolled independently instead. This would remove the jumping of the middle column with the price. Thoughts? |
@startailcoon that's the way it was before, but it doesn't make a lot of sense to scroll independently since the center is where the buy and sell meet. |
Right, but we need to make sure the center stay center and doesnt jump around. Can we achieve that? |
Only had a few intermittent evenings to hit this. I couldn't solve the jump factor consistently without splitting. Pulled out StickyTable, and I have it working, but I will need to confirm it's responsive. |
@mattbeckman can you sneak this into the current sprint? |
Can I claim this? |
@kapeer42 Yes, but check with @startailcoon that it's still needed. I have a branch where StickyTable was pulled out of OrderBook, but OrderBook has been refactored since then, and due to conflicts may be faster to rethink it if the issue still exists. |
StickyTables are still present as before, even on the new Exchange Design, and the current issue is still present on it. Give it your best crack @kapeer42 👍 |
@startailcoon |
@kapeer42 @sschiessl-bcp As it was based on the new exchange design I've cherry picked the code for PR #1875 and merged it to its branch. However, it still seems to be a slight issue with the centering, please see my comment on #1875 (comment) |
@startailcoon I'm working on the issue on latest develop, will push soon |
There is no specific change to that part based on the new exchange design, so I wouldn't have any issue making it work once you have it ready. |
Perfect, thanks! |
1 similar comment
Perfect, thanks! |
#1689 - Fix vertical order book centering
Describe the bug
The vertical orderbook is at times jumpy when the centering isn't updated properly. There is at times a timedelay when the orderbook is populated before the centering is issued.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Centering done without visual delay
Screenshots (LiceCap, wait for it)
![bitshares-pricejumpy](https://user-images.githubusercontent.com/12114550/42535765-9856ccd6-8490-11e8-86c6-2961b6ecf851.gif)
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: