-
-
Notifications
You must be signed in to change notification settings - Fork 861
ER: Refactor Modal From Project and Wins Page #8036
Copy link
Copy link
Open
Labels
Complexity: LargeDraftIssue is still in the process of being createdIssue is still in the process of being createdEREmergent RequestEmergent RequestFeature: Refactor HTMLIssue Making: Level 2Make issue(s) from an ER or EpicMake issue(s) from an ER or Epicmilestone: missingrole: front endTasks for front end developersTasks for front end developerssize: 0.25ptCan be done in 0.5 to 1.5 hoursCan be done in 0.5 to 1.5 hours
Metadata
Metadata
Assignees
Labels
Complexity: LargeDraftIssue is still in the process of being createdIssue is still in the process of being createdEREmergent RequestEmergent RequestFeature: Refactor HTMLIssue Making: Level 2Make issue(s) from an ER or EpicMake issue(s) from an ER or Epicmilestone: missingrole: front endTasks for front end developersTasks for front end developerssize: 0.25ptCan be done in 0.5 to 1.5 hoursCan be done in 0.5 to 1.5 hours
Type
Projects
Status
Ice box
Emergent Requirement - Problem
Redundant code for a modal/overlay on
projectandwinspages.Issue you discovered this emergent requirement in
Date discovered
03/25/25
Did you have to do something temporarily
Who was involved
@dvernon5
@daras-cu
@gmgonzal
What happens if this is not addressed
There will be redundant code, and the modal will not be able to be reused easily in the future.
Resources
Example of modal: Projects Page select "Search Tips" link
Potential approach: #7866 (comment)
Additional Information: #7866 (comment)
Recommended Action Items
Potential solutions [draft]
Create two issues:
Issue 1: Factor out the modal from the
projectpage.current-projects.htmlandsearch-tip-modal.htmlto replace the previous page-specific modal elements with those imported from the newly created files.search-tip-modal.htmlas this does not need to be dynamically generated)Issue 2: Refactor
winspage to use the new modal CSS and JavaScript files, maintaining the mobile-specific functionality.