Skip to content

fix: reflect position prop on modal#38

Merged
johnlcos merged 1 commit intomainfrom
feat/TRAIL__minor_updates
May 1, 2026
Merged

fix: reflect position prop on modal#38
johnlcos merged 1 commit intomainfrom
feat/TRAIL__minor_updates

Conversation

@johnlcos
Copy link
Copy Markdown
Member

@johnlcos johnlcos commented May 1, 2026

Summary

Fixes: #

Related Tickets: TRAIL-

  • Reflect the position prop on the modal so that the conditional styling are properly picked up.

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 📚 Documentation update
  • 🎨 Style/UI change
  • ♻️ Refactor (no functional changes)
  • ⚡ Performance improvement
  • 🧪 Test update
  • 🔧 Build/CI configuration
  • 🧹 Chore (dependency updates, cleanup, etc.)

Changes Made

Primary Changes

Secondary/Collateral Changes


Technical Notes

Files Modified

Implementation Details


Testing

How to Test

Test Coverage

  • Unit tests added/updated
  • Integration tests added/updated
  • E2E tests added/updated
  • Manual testing completed

Browsers Tested

  • Chrome
  • Firefox
  • Safari
  • Edge

Potential Regressions


Screenshots / Videos

Before After

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code in hard-to-understand areas
  • I have updated documentation as needed
  • My changes generate no new warnings or errors
  • I have tested my changes locally
  • Any dependent changes have been merged and published

Additional Context

@johnlcos johnlcos merged commit 59d793a into main May 1, 2026
3 of 4 checks passed
@johnlcos johnlcos deleted the feat/TRAIL__minor_updates branch May 1, 2026 14:33
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