search

INCF / artem-is

https://incf.github.io/artem-is/
Creative Commons Attribution 4.0 International
13 stars 11 forks source link

Conditional Display notification for items currently show #58

Closed suzyjstyles closed 2 years ago

suzyjstyles commented 2 years ago

We currently show a text indicator for hidden questions, but not for unhidden Qs.

This is important because if you change something 'upstream' (e.g., to check what is dependent on an earlier Q) the values are deleted. Perhaps better to include a similar text indicator for dependent visible Qs:

Current: "Conditional questions 69 and 70 are not displayed." Suggested addition: Conditional questions 69 and 70 are displayed due to data entered at question 68

asoskic commented 2 years ago

@dpajicns says it is not trivial to implement this additional information about which question was the one that guided displaying a given conditional question. Moreover the users are already aware of the question's conditional nature - conditional questions never show up until user choses a given value on the previous question, and they always show up immediately below this question, so the users can always see that they have just popped up after choosing an appropriate option, while a second ago there was a notification indicating the existence of hidden conditional questions. With that in mind, it does not seem like adding a notification about which particular question influenced showing or hiding a given question would be worth the effort needed to make this change.

asoskic commented 2 years ago

Edit: there is one case where the users may not be aware of the conditional nature of questions because they are not next to each other, and this is preprocessing software if they select that they did not use the same software for the entire pipeline. However, we have added instructions to clarify this in the "Details" column rather than having to change the code to add the suggested message, which would burden Dejan more.