Strange behaviour of wizard

kbss-cvut / s-forms

SForms - Semantic form generator and processor. For ontology-based smart forms.

GNU Lesser General Public License v3.0

4 stars 7 forks source link

Strange behaviour of wizard #323

Closed blcham closed 6 months ago

blcham commented 6 months ago

In general, we do not want this behavior.

I believe such behavior is useful only in cases where we support incremental question display (i.e., questions would be shown to the user incrementally -- when he/she enters field n, the field n+1 would be shown to him)

A/C:

[x] it is found out if we support incremental question display
[x] default behavior is changed so user can navigate to different tab without "Next" button
[x] If user must use "Next button", the Wizard tab should be grayed out with a message something like: "Navigation to this tab is disabled to prevent overlooking important questions. Use form-related buttons to navigate to this TAB instead."

LaChope commented 6 months ago

In SForms, it is possible to click "FHA classification" without clicking "Next" first.

Also, SForms does not support incremental question display, only "irrelevant" questions (e.g. is-relevant-if).

blcham commented 6 months ago

So is it possible to find out why it is not working in record-manager?

LaChope commented 6 months ago

Actually, we support incremental question. I overlooked enableForwardSkip option. What do you think about renaming it "enableWizardStepSkip", to make it clearer?

blcham commented 6 months ago

I am ok with renaming, but would like to know what it means that we support "incremental questions" :)

LaChope commented 6 months ago

Sorry, "incremental step" would be more appropriate. With this option, we can force user to use "Next" and "Previous" buttons to change the wizard step. IMO, if enableForwardSkip is disabled, it should be at least greyed out and have some tool tip. In this current state, it really looks like it is a bug.

blcham commented 6 months ago

I unchecked default behavior is changed so user can navigate to different tab without "Next" button as I believe it was not solved, leaving this ticket unresolved. To resolve it we need to change SForms.

IMO, if enableForwardSkip is disabled, it should be at least greyed out and have some tool tip

I agree, it should be grayed out with a message something like: "Navigation to this tab is disabled to prevent overlooking important questions. Use form-related buttons to navigate to this TAB instead."

LaChope commented 6 months ago

I agree, it should be grayed out with a message something like: "Navigation to this tab is disabled to prevent overlooking important questions. Use form-related buttons to navigate to this TAB instead."

I added this to A/C.