ER: Update wins.js so the Wins overlay uses icon-github.svg

[roslynwythe]

Emergent Requirement - Problem

The wins overlay (which appears when the user expands a wins card using "See More" link) contains a reference to /assets/images/wins-page/icon-github-small.svg, and we would like to replace that reference with a reference to _includes/svg/icon-github.svg.

Issue you discovered this emergent requirement in

• 6911

Date discovered

12/18/2024

Did you have to do something temporarily

• [ ] YES
• [x] NO

Who was involved

@roslynwythe

What happens if this is not addressed

Resources

• 6911

• WINS HTML: pages/wins/wins.html
• WINS overlay HTML template: _includes/wins-page/wins-overlay.html
• Wins webpage
• making SVGs WCAG compliant

• 6654

• WINS JavaScript: /assets/js/wins.js

Recommended Action Items

• [ ] Make a new issue
• [ ] Discuss with team
• [ ] Let a Team Lead know

Potential solutions [draft]

• The new issue will be similar to #6911 however changes will be made in the function updateOverlay and in the template wins-overlay.html, and the changes are a bit more complex due to more complex JS involved.

[github-actions[bot]]

Hi @roslynwythe.

Please don't forget to add the proper labels to this issue. Currently, the labels for the following are missing:

• Complexity, Role, Feature

NOTE: Please ignore this comment if you do not have 'write' access to this directory.

To add a label, take a look at Github's documentation here.

Also, don't forget to remove the "missing labels" afterwards. To remove a label, the process is similar to adding a label, but you select a currently added label to remove it.

After the proper labels are added, the merge team will review the issue and add a "Ready for Prioritization" label once it is ready for prioritization.

Additional Resources:

• Wiki: How to create issues
• Wiki: How to read and interpret labels

[roslynwythe]

@ExperimentsInHonesty I want to mention that while this issue relates to the GitHub icon, I believe a similar situation exists with the LinkedIn icon. I'm not sure if we should address the LinkedIn icon in this ER or a different one.