unfoldingWord / tc-create-app

translationCore Create
https://create.translationcore.com/
MIT License
0 stars 5 forks source link

Cursor position goes back to previous clicked location when view Column selections are changed #1632

Closed elsylambert closed 6 months ago

elsylambert commented 7 months ago

Describe the bug I. When I reach the bottom of a page, reviewing notes, and click the right arrowhead to turn the page, the cursor jumps to the top usually, but not always, maybe every fifth page or so, I have to push the curser up the slide myself. II. If I click to turn on Quote for a quick look at the quote, the location jumps back to verse 1 (from verse 15). And if I click on Preview to see what the bolding actually looks like, my location in tN jumped from Proverbs 15:15 to 15:4

Steps to reproduce the behavior:

  1. Go to en_tn/File. Scroll to the bottom of the page.
  2. Click on right arrowhead to move over to next page, Cursor jumps to the top but sometimes, it does not.
  3. From the header Menu> View columns> Check/Uncheck Columns
  4. Note that cursor position moved few pages back.
  5. Click on Preview button. Note that cursor position moved few pages back.

Expected behavior Cursor position should be at the active row when columns added into view.

RevantCI commented 7 months ago

Hi @elsylambert We reviewed the bug and code and found a few related issues and comments regarding the same workflow.

  1. 1088

  2. 1089

  3. 1395

  4. 1606

  5. 1607

  6. 1614

Though we will just clear the doubts we had regarding it here.

Current workflow

  1. If we click on a row to edit it, its position is saved
  2. On clicking Preview/ changing columns / closing search it goes back to the clicked row
  3. The above actions takes place irrespective of the current position. That is if its on the same page or if we navigate to other pages using the left and right arrow at the bottom page navigation menu

Proposed changed workflow

Else if you had another workflow in mind please let us know via the comments or we can have a call to discuss it.

Note: 2 of the above actions i.e. preview and clear search both navigate to the last clicked row and highlight it blue. The column change only navigates to the last clicked row but doesn't highlight it in blue. as per the comment in this PR.

elsylambert commented 7 months ago

@RevantCI The current workflow you mentioned is what we expect to happen. Current workflow:

RevantCI commented 7 months ago

Hi @elsylambert Both the bugs you had mentioned in your last comment are fixed and the datatable-translatable RCL is updated to 1.2.4 as per our call and in TC create the version has been bumped to the same 1.2.4. Also as you had mentioned we have updated the version number of TC create itself to 1.17 for the release.

elsylambert commented 7 months ago

Thanks @RevantCI

elsylambert commented 7 months ago

Looks good in translationCore Create - v1.17 build 277-90c96ed. Cursor positions stays on the active row when Preview is clicked, columns selections are changed or search is closed.