Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 2, 2025

No description provided.

Copilot AI and others added 2 commits November 2, 2025 06:34
Co-authored-by: danielchalmers <7112040+danielchalmers@users.noreply.github.com>
…nd performance optimizations

Co-authored-by: danielchalmers <7112040+danielchalmers@users.noreply.github.com>
Copilot AI changed the title [WIP] Add animations for calendar and UI elements Add Material Design 3-inspired animations throughout the app Nov 2, 2025
Copilot AI requested a review from danielchalmers November 2, 2025 06:40
Copilot finished work on behalf of danielchalmers November 2, 2025 06:40
@danielchalmers
Copy link
Owner

@copilot

  1. Color changes should happen in sync with each other
  2. The snackbar animation is glitchy. Just use the default animation (revert your changes)
  3. Dialog open animation is slow and doesn't feel like Material Design 3
  4. Add animations for adding or removing elements/medications
  5. Add animations for moving / changing the order of elements/medications
  6. Remove all new hover/focus/press animations except days in calendars which need to animate the entire box on press
  7. Add transitions when changing calendar years or index days or any other "switcher"
  8. Make page load transitions more subtle

…vert snackbar, remove hover animations except calendar press, add subtle page transitions

Co-authored-by: danielchalmers <7112040+danielchalmers@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 2, 2025

@copilot

  1. Color changes should happen in sync with each other
    ...

I've addressed all 8 points in commit e819e55:

  1. ✅ Color transitions now sync globally with ease-in-out (0.2s) on all background-color, color, and border-color properties
  2. ✅ Snackbar animation reverted to MudBlazor default
  3. ✅ Dialog animation now faster (0.15s instead of 0.3s) with tighter scale (0.95→1.0) and MD3 easing cubic-bezier(0.2, 0, 0, 1)
  4. ✅ Elements/medications fade in when added (0.2s ease-out with translateY)
  5. ✅ Elements/medications have smooth transitions when reordered (0.2s ease-in-out on all properties)
  6. ✅ Removed all hover/focus animations except calendar day press (0.1s scale to 0.95)
  7. ✅ Calendar switchers (year/day) have smooth transitions (0.2s ease-in-out)
  8. ✅ Page load animation more subtle (0.15s fade-in-up with only 8px translateY instead of 20px)

The breathing animation for the current calendar day remains at 3s infinite.

Copilot finished work on behalf of danielchalmers November 2, 2025 16:34
@danielchalmers
Copy link
Owner

@copilot Revert all animations except keeping transitions for pages and switching. This PR will only focus on that

Co-authored-by: danielchalmers <7112040+danielchalmers@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 2, 2025

@copilot Revert all animations except keeping transitions for pages and switching. This PR will only focus on that

Reverted all animations in commit 77691f4. The PR now only includes:

  1. Page transitions: 0.15s fade-in-up animation (8px translateY) on .page-body
  2. Switcher transitions: 0.2s ease-in-out on .switcher and .switcher-header (for calendar year/day navigation)

All other animations (calendar days, dialogs, loading spinners, categories, etc.) have been removed.

Copilot AI changed the title Add Material Design 3-inspired animations throughout the app Add page and switcher transitions Nov 2, 2025
Copilot finished work on behalf of danielchalmers November 2, 2025 16:52
@danielchalmers danielchalmers changed the title Add page and switcher transitions Add page fade-in transition Nov 2, 2025
@danielchalmers
Copy link
Owner

this hides some of the jank caused by blazor scroll bug

@danielchalmers danielchalmers marked this pull request as ready for review November 2, 2025 17:07
@danielchalmers danielchalmers merged commit 104689a into main Nov 2, 2025
1 check passed
@danielchalmers danielchalmers deleted the copilot/add-calendar-animations branch November 2, 2025 17:07
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