-
Notifications
You must be signed in to change notification settings - Fork 3
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
Issue 198 repair attempt full screen modal #232
base: main
Are you sure you want to change the base?
Conversation
…orm to components, adjusted names to not be conflicting
…fit modal, renamed unused repairAttemptForm
…into issue-198-Repair_attempt_full-screen_modal
…into issue-198-Repair_attempt_full-screen_modal
…into issue-198-Repair_attempt_full-screen_modal
…into issue-198-Repair_attempt_full-screen_modal
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…into issue-198-Repair_attempt_full-screen_modal
…into issue-198-Repair_attempt_full-screen_modal
It's still a modal but it takes about the full width of the screen, is that good or should it be a separate page on mobile view? The modal uses headlessui's dialog component and it seems that cant take up the whole screen. |
Change Summary
[Briefly summarise the changes that you made. Just high-level stuff]
Change Form
Fill this up (NA if not available). If a certain criteria is not met, can you please give a reason.
Other Information
[Is there anything in particular in the review that I should be aware of?]