DCS-LCSR / SignStream3

Sign language linguistics annotation software
2 stars 0 forks source link

Display non-empty fields #686

Open cneidle opened 10 months ago

cneidle commented 10 months ago

On opening an utterance, by default the non-empty fields (and only those) should be displayed.

So, I click on the non-empty fields button, and that is fine.

But when I delete one item, the field display reverts back to showing all sorts of empty fields.

Screen Shot 2023-08-31 at 4 15 57 PM

When I delete the item highlighted above in red, this happens:

Screen Shot 2023-08-31 at 4 16 11 PM

When I click on the non-empty fields button, it returns to the proper display

Screen Shot 2023-08-31 at 4 16 19 PM
cneidle commented 10 months ago

The above is using SignStream 3.4.1 on Mac OS 11.7.9. The file was DSP_Sentences-DL.

cneidle commented 9 months ago

This is a serious issue, but it is a little bit complicated; we should discuss.

gregorydimitriadis commented 9 months ago

Main issue at hand for this: do not trigger the default view on a delete event. This is what's occurring now and resulting in the inconsistency.

gregorydimitriadis commented 9 months ago

Carol, to clarify this portion: "by default the non-empty fields (and only those) should be displayed."

If we were to do that, would that apply to nonmanual/gloss/gesture alike? Or just nonmanuals specifically.

cneidle commented 9 months ago

I would think that our current display of basic gloss/gesture fields should remain as it is now.

For optionally specified information about gestures (e.g., location, function, etc.), then it should be like the non-manuals, in that the default display should be non-empty, with the option of clicking on the button to display those that are selected for display.

Thanks! Carol

On Oct 5, 2023, at 3:02 PM, Greg Dimitriadis @.***> wrote:

Carol, to clarify this portion: "by default the non-empty fields (and only those) should be displayed."

If we were to do that, would that apply to nonmanual/gloss/gesture alike? Or just nonmanuals specifically.

— Reply to this email directly, view it on GitHub https://github.com/DCS-LCSR/SignStream3/issues/686#issuecomment-1749482416, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADH7NQQDQAMMGWDJ4CL5NXLX5375XAVCNFSM6AAAAAA4GTCF66VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBZGQ4DENBRGY. You are receiving this because you authored the thread.

cneidle commented 7 months ago

Non-empty fields are not currently displayed by default. I open this utterance, for example:

Screen Shot 2023-11-30 at 11 30 45 AM

The hp:jut and hm:nod fields are displayed although they are empty.

The display non empty button does work properly, however. And editing fields does not affect the display non empty selection... So that much is good!