Skip to content

Fix/routine builder modal position#777

Open
Sonalis1299 wants to merge 3 commits into
aryandas2911:mainfrom
Sonalis1299:fix/routine-builder-modal-position
Open

Fix/routine builder modal position#777
Sonalis1299 wants to merge 3 commits into
aryandas2911:mainfrom
Sonalis1299:fix/routine-builder-modal-position

Conversation

@Sonalis1299
Copy link
Copy Markdown

📌 Description

Fixed the modal positioning issue in the Routine Builder section where the “Create New Task” and “Save Routine” modals were appearing relative to the scrollable webpage instead of the visible viewport.

The issue was caused by the parent container animation class affecting the modal’s fixed positioning behavior.

🔗 Related Issue

Closes #732

🛠 Changes Made

  • Removed the conflicting animation class from the parent wrapper
  • Fixed modal viewport positioning behavior
  • Ensured modals stay centered regardless of page scroll position
  • Improved modal usability and user experience

📸 Screenshots

  • Before
Screenshot 2026-05-19 084802
  • After
Screenshot (524)

✅ Checklist

  • Code runs locally
  • Followed project structure
  • No console errors
  • Properly tested changes
  • Linked the issue

🚀 Notes for Reviewers

Tested the modals after scrolling the page vertically. Both “Create New Task” and “Save Routine” modals now remain centered relative to the viewport without requiring additional page scrolling.

@Sonalis1299
Copy link
Copy Markdown
Author

Hi @aryandas2911 ,I have completed the fix for the modal viewport positioning issue in the Routine Builder section and submitted the PR. Kindly review it when possible. Thank you!

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.

Routine Builder modals are not centered in viewport

1 participant