Skip to content

fix : replaced alert with inline error in TaskPreview.jsx duration validation - #2052

Open
tmdeveloper007 wants to merge 1 commit into
aryandas2911:mainfrom
tmdeveloper007:#2047
Open

fix : replaced alert with inline error in TaskPreview.jsx duration validation#2052
tmdeveloper007 wants to merge 1 commit into
aryandas2911:mainfrom
tmdeveloper007:#2047

Conversation

@tmdeveloper007

Copy link
Copy Markdown

Summary of What Has Been Done

Replaced the browser alert() call in the TaskPreview component duration modal with a state-based inline error message for better UX.

Changes Made

  • Added durationError state to the TaskPreview component
  • Replaced alert() with setDurationError() when validation fails
  • Cleared error on input change and modal close
  • Added inline error display below the duration input

Impact it Made

  • Eliminates jarring browser dialog
  • Works properly on mobile
  • Consistent with the app's UX patterns

Note: Please assign this PR to the tmdeveloper007 account.

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.

1 participant