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

Adjust Preview Event When Changing Times in Form #46

Open
1 task done
tyler-dane opened this issue Dec 19, 2023 · 0 comments · May be fixed by #106
Open
1 task done

Adjust Preview Event When Changing Times in Form #46

tyler-dane opened this issue Dec 19, 2023 · 0 comments · May be fixed by #106
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@tyler-dane
Copy link
Contributor

tyler-dane commented Dec 19, 2023

Prerequisites

  • Using an up-to-date main branch

Expected Behavior

As a user, when changing an event's time in the event form, I should see the preview event rectangle size and time values change immediately

Current Behavior

Screen Shot 2023-12-19 at 7 47 14 AM

  • Notice: The form's start time is different from the rectangle's start time

When changing an event's time from the form, the preview rectangle and time values don't change until after submitting the event

Steps to Reproduce

  1. Create new event

  2. Edit event by clicking it to open the form

  3. Change start/end dates using the form's input picker

  4. Notice: The form's time values are different from the preview event's

Possible Solution (Not obligatory)

Coordinate state between EventForm.tsx and useDraftUtil.ts in a similar way as the title and priority states

  • The title and priority values are updating in the preview correctly, so try using that same approach for the start and end time values

Context

@tyler-dane tyler-dane added bug Something isn't working good first issue Good for newcomers labels Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
Development

Successfully merging a pull request may close this issue.

1 participant