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

Final Day Planner Adjustments #83

Closed
ThatMegamind opened this issue Apr 15, 2024 · 0 comments · Fixed by #91
Closed

Final Day Planner Adjustments #83

ThatMegamind opened this issue Apr 15, 2024 · 0 comments · Fixed by #91
Assignees
Labels

Comments

@ThatMegamind
Copy link
Collaborator

Part 1: Toast Location

  • At the moment, toasts on the Day Planner appear slightly below the add event button, but they should appear closer to the top right side of the Day Planner

Now:
Screenshot 2024-04-15 at 2 16 31 PM

High Fidelity:
Screenshot 2024-04-15 at 2 17 45 PM

Part 2: Add Existing Event Icon Button

  • The designers would like the button to add an existing event from the Catalog to the Published Schedule to be slightly darker and larger
  • Check with the high fidelity to clarify the exact changes

Now:
Screenshot 2024-04-15 at 2 20 21 PM

High Fidelity:
Screenshot 2024-04-15 at 2 23 48 PM

Part 3: Edit Date Bug

  • Editing the date via the Day Planner causes a bug where the date that displays in the Day Planner is 1 day behind the correct date (seen in the edit day modal)— this bug should be fixed

Example:
Screenshot 2024-04-15 at 2 26 13 PM

Part 4: New Season Rendering Issue

  • Upon creating a new day that also creates another season, it does not rerender the published schedule page. The new season does not show up in the dropdown unless you refresh the page.
  • Update the code to ensure that this change is rendered without having to refresh the page
@chloecheng8 chloecheng8 self-assigned this Apr 18, 2024
@cherhchen cherhchen self-assigned this Apr 18, 2024
@ThatMegamind ThatMegamind linked a pull request Apr 27, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants