Giters

kbss-cvut / s-forms

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

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Strange behaviour of wizard

blcham opened this issue · comments

commented

image

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:

  • it is found out if we support incremental question display
  • default behavior is changed so user can navigate to different tab without "Next" button
  • 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."
commented

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).

commented

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

commented

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

commented

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

commented

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.

commented

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."

commented

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.