DCS-LCSR / SignStream3

Sign language linguistics annotation software
1 stars 0 forks source link

UNDESIRABLE UI CHANGES from previous version: 1) Database Window Toolbar #466

Open cneidle opened 3 years ago

cneidle commented 3 years ago

If possible, can you please swap the icons (colors) for Display Non-empty and Refresh Default Display? That way the same icon used previously for Display Non-empty will be used in the new version for the same thing. I am drafting documentation, and it seems silly to have to explain that the previous Display Non-Empty icon now is used for something else, and a different icon is used for Display Non-Empty.

I have to say that I also feel silly having to explain this:

For consistency with the options available in the Utterance Window, there are sets of buttons available in the Database Window for both Event (7 in Figure 1) and Entity (8 in Figure 1), although at the Database View level, both correspond to Utterance. So, either set of buttons can be used with reference to an Utterance. The “Video” label (2 in Figure 2) has been replaced by a label for “Range” (9 in Figure 1) – which refers to the entire video.

If it were up to me, we would have a single grouping (not Event or Entity, but Utterance), and we would change "Range" to "Video". The toolbars are not otherwise identical for Database and Utterance view, so we should make this maximally transparent, as far as I'm concerned. But I was overruled on this. SO I will need to have these explanations in the documentation.

Thanks.

foltik commented 3 years ago

Colors have been swapped for next version.

cneidle commented 3 years ago

Thank you

On May 28, 2021, at 9:46 AM, Jack Foltz @.***> wrote:

Colors have been swapped for next version.

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

douglas-motto-at-rutgers commented 3 years ago

Leaving the rest open as future change.

cneidle commented 3 years ago

I have to say I am unhappy about this. This is unnecessarily user unfriendly. It seems absurd that we have to wait until version 3.8 to be able to announce that we finally have 'Play Utterance' labeled as such in the Database View. It likewise seems unfortunate that we have to begin our documentation about the purported improvements in the new version with explanations about how and why the toolbar for the database view has become more obscure. If it is too difficult to relabel "event" as "utterance" for this window (which would be an improvement relative to the previous version), is it at least possible to hide the "entity" grouping here, which is a new addition that serves no purpose at all here, and unnecessarily introduces confusion ??

cneidle commented 3 years ago

This is not correct:

> Colors have been swapped for next version.

And I do not consider it an enhancement to maintain the same icon or Display Non-Empty that was used in previous versions

This should be fixed prior to next release.

douglas-motto-at-rutgers commented 3 years ago

It looks like the color change was implemented and deployed in v4 but not merged into v3.

douglas-motto-at-rutgers commented 3 years ago

As of

SignStream3_3.7.0_pre_2021_06_01_1624_7ae614a9

colors in v3 switched.

cneidle commented 3 years ago

I don't see how this is an enhancement. It is reversing a change made in this version that is undesirable...

cneidle commented 2 years ago

See also:

2) MENU ITEMS that do not belong in this version #484 3) DISPLAY TIME PERIOD - either implement or eliminate the reference to it in the interface #147 , #420