-
Notifications
You must be signed in to change notification settings - Fork 82
Follow-up question to select one seems broken #166
Comments
Hey Florian; always good to hear from you. You're right, something is goofy about the enable/disable state of the select box. I'm working on it. To resolve that warning, you'll have to disable "Display on one screen" on the enclosing group. It's a limitation of Collect; it can't (doesn't bother to) reprocess the relevance logic while the whole group is already displayed. |
Can we re-open this? I believe the issue of 'select question shown as invalid in group' issue is still not addressed. From OP: Select one inside groups are showing up as "invalid" no matter what I do with the follow up question. I see the inline error message "Because this control is within a single-screen group (field list), any expressions that reference other fields in the same group will not work." (props on all of the new changes btw - lots of great features added recently!) |
hey colin: that's unfortunate. does it happen with all your forms? and—obvious question, i'm sorry, but are you sure none of the parent groups are marked as "Display On One Screen"? |
Thanks Issa.
Re: Obvious question - yeah, the parent groups are marked as 'display on
one screen' (sorry, I should've been explicit about this).
So perhaps this is more of an issue of 'how should it work'? I feel like a
display-on-one-screen-group of [select one + text] should not default to
generating a warning/error - there's nothing linking those two questions
per se (eg the text is not a follow up to the select).
My use case is such - I have a select one with a bunch of options, the last
of which is 'Other'
Then I have a text box with something like 'If Other selected, please
specify'
I would prefer the 'If Other, please specify' text to *not* appear on a
different screen (since sometimes the Select provides contextual clues for
remembering how to fill in the Specify).
I have been using this pattern since the dawn of ODK, so perhaps I'm stuck
in my way of thinking?
…On Wed, Jan 9, 2019 at 8:09 AM Issa Tseng ***@***.***> wrote:
hey colin: that's unfortunate. does it happen with all your forms?
and—obvious question, i'm sorry, but are you sure *none* of the parent
groups are marked as "Display On One Screen"?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#166 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAuEmKr_aK_2OOAYVLAgriEl6epLAxJjks5vBTNWgaJpZM4ODMwY>
.
--
Colin McCann
|
unfortunately, the restriction is due to how XForms/JavaRosa/ODK Collect work. the follow-up question functionality shows an additional text question only when some particular select option is chosen. but Collect only decides whether or not to display a field at the time the field is presented. when "display on one screen" is selected, all fields are presented at once, and then the display logic is never reconsidered unless the entire group is navigated away from and then back to. this means the follow-up question won't display when you'd expect, which is why we prevent the combination. if your intention is to always display a text field that has a label teaching the user when to/when not to fill it out, then i would suggest just adding a plain text question after the select question rather than using the follow-up question feature. |
Ok, makes sense. However, I may have not been sufficiently clear.
To you point:
"if your intention is to *always* display a text field that has a label
teaching the user when to/when not to fill it out, then i would suggest
just adding a plain text question after the select question rather than
using the follow-up question feature."
That is indeed what I am doing. The two questions aren't linked. That is, I
haven't checked the 'follow-up question' box. See below (hopefully inline
photos work here):
[image: Screen Shot 2019-01-09 at 8.50.28 AM.png]
…On Wed, Jan 9, 2019 at 8:37 AM Clint Tseng ***@***.***> wrote:
unfortunately, the restriction is due to how XForms/JavaRosa/ODK Collect
work.
the follow-up question functionality shows an additional text question
*only* when some particular select option is chosen.
but Collect only decides whether or not to display a field at the time the
field is presented. when "display on one screen" is selected, all fields
are presented at once, and then the display logic is never reconsidered
unless the entire group is navigated away from and then back to.
this means the follow-up question won't display when you'd expect, which
is why we prevent the combination.
if your intention is to *always* display a text field that has a label
teaching the user when to/when not to fill it out, then i would suggest
just adding a plain text question after the select question rather than
using the follow-up question feature.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#166 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAuEmLo9o6koLVyMFaDC2lgcP4Hxy0l-ks5vBTmzgaJpZM4ODMwY>
.
--
Colin McCann
|
photo didn't work. :) |
ah, that's definitely a bit goofier. |
apparently long ago when i implemented validations i got partially done with implementing a warning-level severity but never fully surfaced it. the intention with this validation check was to surface a warning, not an error. so the issue is not that the error should not appear, but rather that it should not be so scary. |
* apparently long ago when i implemented validations i got partially done with implementing a warning-level severity but never fully surfaced it. * and in particular, the field list expression check was meant to be a warning, not an error. it's supposed to show, but it's not supposed to look so scary. * so here i finish implementing that.
…d error. * in the relevance case we don't know if the expression refers within the group. * but in the followup question case we do and it will never work (at least in odk collect). * i guess non-collect-users just need to ignore the error..?
* apparently long ago when i implemented validations i got partially done with implementing a warning-level severity but never fully surfaced it. * and in particular, the field list expression check was meant to be a warning, not an error. it's supposed to show, but it's not supposed to look so scary. * so here i finish implementing that.
* apparently long ago when i implemented validations i got partially done with implementing a warning-level severity but never fully surfaced it. * and in particular, the field list expression check was meant to be a warning, not an error. it's supposed to show, but it's not supposed to look so scary. * so here i finish implementing that.
* in the relevance case we don't know if the expression refers within the group. * but in the followup question case we do and it will never work (at least in odk collect). * i guess non-collect-users just need to ignore the error..?
fixed in #217. |
I'm having a few troubles with the current "select one":
A MWE:
choice_one
shows up as invalid in ODK Build, choice_two works.Uploaded to an ODK Aggregate server and filled in in ODK Collect, the behaviour is:
It would be awesome if the follow-up could be ignored inside groups (maybe the "invalid" status is a false positive?), and if the follow-up dropdown could be active when follow up is selected (bug?).
The text was updated successfully, but these errors were encountered: