Skip to content

release/DEV-819: Prevent swiping through slides with error messages#873

Merged
farhantariq12b merged 3 commits intoproductionfrom
release/DEV-819
Feb 19, 2026
Merged

release/DEV-819: Prevent swiping through slides with error messages#873
farhantariq12b merged 3 commits intoproductionfrom
release/DEV-819

Conversation

@farhantariq12b
Copy link
Contributor

@farhantariq12b farhantariq12b commented Feb 17, 2026

DEV-819: The user can swipe through slides with error message on the screen

Parent ticket: DEV-473 - Single submission logic

Changes

  • DEV-819 Disable swiper navigation (next, prev, touch) when a blocking error toast (duplicate submission or offline message) is shown, preventing users from swiping through slides while error messages are on screen
  • DEV-819 Reload the page when user clicks OK on the submission-exists toast, unless a redirect action is configured
  • DEV-819 Add hasBlockingError flag to persist swipe-disabled state through touchmove events so the swiper stays locked even after touch interactions

Files Changed

  • js/libs/form.js — Added swiper disable logic in showSubmissionExistMessage() and showOfflineMessage(), added page reload on OK dismiss, and integrated hasBlockingError check into the touchmove swiper validation

Test Plan

  • Multi-step form with single submission: submit once, trigger duplicate submission toast — verify slides cannot be swiped
  • Click OK on the duplicate submission toast — verify page reloads
  • Multi-step form with single submission + redirect configured — verify OK does NOT reload but follows redirect
  • Offline scenario: trigger offline toast on multi-step form — verify slides cannot be swiped
  • Normal multi-step form (no errors): verify swiping still works based on validation state
  • Vertical scroll on slide component — verify it doesn't interfere with swipe lock

🤖 Generated with Claude Code

… the screen (#862)

* DEV-819 [FEAT] User can't swipe through slides with error messages on the screen

* Fixed the swiper disabling logic
@farhantariq12b farhantariq12b changed the base branch from master to production February 17, 2026 13:17
@farhantariq12b farhantariq12b changed the title DEV-819: Form builder - Prevent swiping through slides with error messages release/DEV-819: Prevent swiping through slides with error messages Feb 17, 2026
Copy link
Collaborator

@galisufyan-327 galisufyan-327 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@farhantariq12b farhantariq12b merged commit dd0d66e into production Feb 19, 2026
2 of 3 checks passed
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

Successfully merging this pull request may close these issues.

2 participants