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

NQ of CellMove incorrectly processed by the client #341

Open
mkromberg opened this issue Jan 4, 2025 · 1 comment
Open

NQ of CellMove incorrectly processed by the client #341

mkromberg opened this issue Jan 4, 2025 · 1 comment
Milestone

Comments

@mkromberg
Copy link
Collaborator

Start GAMA, select Shares|Market|Prices.

Click on the arrowhead to scroll right. Logging is turned on during the callback processing. Click one more time to get a complete set of log messages during the processing of the event, for example:

04:24.366 R: #3: {"Event":{"EventName":"Scroll","EventID":"30bb71f1-9415-4ddb-a89f-d1386157fe0a","ID":"F0000.App.⍙3S36.Y1","Info":[1,2]}}
04:24.370 T: #3: {"WG":{"ID":"F0000.App.⍙3S36.Y0","Properties":["CurCell"],"WGID":"29"}}
04:24.379 R: #3: {"WG":{"ID":"F0000.App.⍙3S36.Y0","Properties":{"CurCell":[1,1]},"WGID":"29"}}
04:24.383 T: #3: {"WG":{"ID":"F0000.App.⍙3S36.Y1","Properties":["Thumb"],"WGID":"30"}}
04:24.402 R: #3: {"WG":{"ID":"F0000.App.⍙3S36.Y1","Properties":{"Thumb":1},"WGID":"30"}}
04:24.416 T: #3: {"NQ":{"Event":"CellMove","ID":"F0000.App.⍙3S36.Y0","Info":[1,2,1],"NoCallback":0}}
04:24.419 T: #3: {"WS":{"ID":"F0000.App.⍙3S36.Y0.C1","Properties":{"FCol":0}}}
04:24.422 T: #3: {"NQ":{"Event":"GotFocus","ID":"F0000.App.⍙3S36.Y0","Info":["F0000.App.⍙3S36.Y1"],"NoCallback":0}}
04:24.426 T: #3: {"EC":{"EventID":"30bb71f1-9415-4ddb-a89f-d1386157fe0a","Proceed":0}}

Notice:

  1. Although the Server sends:
    {"NQ":{"Event":"CellMove","ID":"F0000.App.⍙3S36.Y0","Info":[1,2,1],"NoCallback":0}}
    The current cell is not moved to [1,2] and no message is echoed back to the server even though the CellMove event is defined (and this was working until recently).

  2. Although the position of the Thumb is reported as 1, the Thumb does not seem to be at the left:
    image

@mkromberg mkromberg added this to the Blocking milestone Jan 4, 2025
@mkromberg mkromberg changed the title NQ of CellMove is not processed by the client NQ of CellMove incorrectly processed by the client Jan 4, 2025
@mkromberg
Copy link
Collaborator Author

Following the above, if you click in cell [1,2] and then click again on the right scroll arrow, you will see (selected messages included):

{"Event":{"EventName":"Scroll","EventID":"5b043049-28af-44ac-a862-9c83820577e2","ID":"F0000.App.⍙3S36.Y1","Info":[1,3]}}
{"WG":{"ID":"F0000.App.⍙3S36.Y0","Properties":{"CurCell":[1,2]},"WGID":"34"}}
{"WG":{"ID":"F0000.App.⍙3S36.Y1","Properties":{"Thumb":2},"WGID":"35"}}
{"NQ":{"Event":"CellMove","ID":"F0000.App.⍙3S36.Y0","Info":[1,3,1],"NoCallback":0}}
{"EC":{"EventID":"5b043049-28af-44ac-a862-9c83820577e2","Proceed":0}}

Note that the server rejects the scroll, it will set the scroll thumb when the NQ'd CellMove is returned to the server (but as mentioned above, this does not happen). The Thumb was set to 2 when the user clicked in [1,2] but the visible thumb did not move.

If you now click on the right scroll arrow AGAIN:

{"Event":{"EventName":"Scroll","EventID":"ae9044db-a7ab-4ff6-918e-52ae8511a623","ID":"F0000.App.⍙3S36.Y1","Info":[1,3]}}
{"WG":{"ID":"F0000.App.⍙3S36.Y0","Properties":{"CurCell":[1,3]},"WGID":"36"}}
{"WG":{"ID":"F0000.App.⍙3S36.Y1","Properties":{"Thumb":2},"WGID":"37"}}
{"NQ":{"Event":"CellMove","ID":"F0000.App.⍙3S36.Y0","Info":[1,4,1],"NoCallback":0}}
{"EC":{"EventID":"ae9044db-a7ab-4ff6-918e-52ae8511a623","Proceed":0}}

Notice that CurCell is reported as [1,3] but it is still [1,2] that is selected in the grid. Also notice that the current cell now changes to [1,3] although the NQ's CellMove was to [1,4].

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

No branches or pull requests

1 participant