Update Modal use throughout the `openedx` GH org to handle required `isOverflowVisible`
Open
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 140
- Forks
- 100
- Avg merge
- 1h 3m
- Merged PRs (30d)
- 30
Description
isOverflowVisible is required as of v22.17.0
- Org-wide search for modal imports from paragon (https://github.com/search?q=org%3Aopenedx+%2Fimport.*%5BMm%5Dodal.*from%5Cs%2B%5B%27%22%5D%40openedx%5C%2Fparagon%5B%27%22%5D%2F&type=code)
Can likely reference the design tokens MFE PRs to see which value to set modals to (since it is a required prop in v23)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the linked org-wide code search for Modal imports from @openedx/paragon, then review the design tokens MFE PRs and platform-roadmap issue 393 for guidance on the prop value. Update the identified Modal usages across the openedx organization to handle required isOverflowVisible, and confirm no searched usage remains incompatible with Paragon v22.17.0 and v23.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, react
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100