Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove fastForward button + overview only navigating backward #129

Closed
QRuhier opened this issue Apr 19, 2024 · 8 comments
Closed

remove fastForward button + overview only navigating backward #129

QRuhier opened this issue Apr 19, 2024 · 8 comments

Comments

@QRuhier
Copy link
Contributor

QRuhier commented Apr 19, 2024

short-term version of #174

  • remove fastForward, but let possibility of returning when user left when he opens the questionnaire
  • menu navigation : seq/subseq must be clickable only if its first page is before the current page
@QRuhier QRuhier changed the title Suite de l'entretien : renvoyer vers la prochaine page nécessitant une réponse FastForward : redirect to the next page requiring a response Apr 19, 2024
@QRuhier
Copy link
Contributor Author

QRuhier commented Apr 23, 2024

lunatic-related subject, to discuss

@QRuhier
Copy link
Contributor Author

QRuhier commented Apr 29, 2024

Also think about the overview navigation (reachable items)

@AnneHuSKa
Copy link

Proposition :

  • Modifier le principe "Suite de l’entretien" en "Dernière page accédée", uniquement proposé à l’ouverture du questionnaire
  • Navigation latérale vers l’arrière uniquement
  • Renommer "suite de l’entretien" en « Dernière page accédée » (sauf si vous avez mieux ?)

@QRuhier
Copy link
Contributor Author

QRuhier commented Jul 5, 2024

  • after discussing : see for removing fastforward button, replaced by a modal (like for Stromae) when entering the questionnaire

  • also change navigation using overview : item should be reachable only if its page is before current page

@QRuhier
Copy link
Contributor Author

QRuhier commented Jul 8, 2024

for overview : see InseeFr/Lunatic#1054

@QRuhier
Copy link
Contributor Author

QRuhier commented Jul 8, 2024

@AnneHuSKa @romaintailhurat in the end we would have for now :

currentPage

instead of saving lastReachedPage as the surveyUnit currentPage, we save the current page (like Stromae)

fastForward

We remove the fastForward button.

We add a modal entering the questionnaire (similar as Stromae) for going back where user left :

  • modal is not present on readonly (review, visu)
  • modal is only present if the currentPage existed and was not the first page
  • one button for returning to 1st page, one button for going back to currentPage (page on which the user was before he left)

menu (overview)

We can only navigate to a seq/subseq which first page <= the current page (before it was compared to the lastReachedPage)

⚠️ For surveyUnits that have already bien started, currentPage was potentially taking value after empty page, so the modal will still redirect to far

@AnneHuSKa
Copy link

@QRuhier : it seems very clear et I think it is ok.
We will mention the last "be careful" to MOAE.

@QRuhier QRuhier changed the title FastForward : redirect to the next page requiring a response remove fastForward button + overview only navigating backward Jul 9, 2024
@AnneHuSKa
Copy link

  • we remove the fastForward button.
  • We add a modal entering the questionnaire (similar as Stromae) for going back where user left :
  • modal is not present on readonly (review, visu)
  • modal is only present if the currentPage existed and was not the first page
  • one button for returning to 1st page, one button for going back to currentPage (page on which the user was before he left)
  • We can only navigate to a seq/subseq which first page <= the current page

ok !

@QRuhier QRuhier closed this as completed Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants