unfoldingWord / gateway-edit

Book Package harmonized view.
https://gatewayedit.com
MIT License
1 stars 4 forks source link

Move Add Button in TWL to Action Row #542

Closed kintsoogi closed 9 months ago

kintsoogi commented 1 year ago

The add TSV button for the Translation Word List resource should be within the table, not within the card header

Example Private Zenhub Image

kintsoogi commented 11 months ago

Describe what your pull request addresses

Test Instructions

elsylambert commented 10 months ago

@kintsoogi is this change merged in Develop branch?

kintsoogi commented 10 months ago

@kintsoogi is this change merged in Develop branch?

Yes it is!

elsylambert commented 10 months ago

v2.1.0 build f845565 QA on https://develop--gateway-edit.netlify.app/. Seeing a number of problems:

  1. When we click on TWL row + to add a row, the add row dialog shows a different verse in Reference and the Original Quote.

Private Zenhub Image

Private Zenhub Image

  1. When in OT books, the Original Quote field is empty. Private Zenhub Image

  2. After we click on Add on Add row dialog, the changes are saved in a different verse due to problem mentioned in Step 1.

birchamp commented 10 months ago

@elsylambert @kintsoogi we should only fill in the new reference, new id, occurrence, and a dropdown for the TWs.

birchamp commented 10 months ago

@elsylambert @kintsoogi let's break out the dropdown as a separate issue.

elsylambert commented 10 months ago

@birchamp I rewrote the issue #506 and moved to Backlog. This issue defines the drop-down for new rows for TWLs.

kintsoogi commented 10 months ago

@danielklapp @elsylambert

This fix is ready in develop now!

Describe what your pull request addresses (Please include relevant issue numbers):

Please include detailed Test instructions for your pull request:

danielklapp commented 10 months ago

When clicking on Add Tsv Row button under Actions, the add row dialog does show correct verse in Reference and the Original Quote. Looks good with v2.1.0 build 65fd995 QA.

elsylambert commented 10 months ago

Add row in TWL looks good in v2.1.0 build 19c7052 QA. The newly added row shows correct reference and saves in the right resource and Reference. Note: However, every new row we add, the add row Modal shows the very first TWL rc link in the file for e.g if Paul is the first TW in the TSV file, rc link rc://*/tw/dict/bible/names/paul ia autofilled in TWL link field. So every time a new row is added, user must type in TWL link in order to add a different word other than Paul. This issue would be addressed by #506 when drop-down for Links would be made available.