Issue #7899 fix srp for assignment

[WavesJ99]

Closes #7899

Describe your changes: I am working on a team and trying to contribute to this project. This first contribution is supposed to be small and get us used to creating a pr. The change basically decouples the selection.js module with the DOM manipulation. This also makes selction.js adhere to SRP. We understand this is not a significant change but that is exactly what we we were looking for because of the description of our assignment.

All Submissions:

• Have you followed the guidelines in our [Contributing document] Yes (I think so)(https://github.com/nasa/openmct/blob/master/CONTRIBUTING.md)?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change? Yes
• Is this a notable change that will require a special callout in the release notes? For example, will this break compatibility with existing APIs or projects that consume these plugins? No

Author Checklist

• Changes address original issue?
• Tests included and/or updated with changes?
• Has this been smoke tested?
• Have you associated this PR with a type: label? Note: this is not necessarily the same as the original issue.
• Have you associated a milestone with this PR? Note: leave blank if unsure.
• Testing instructions included in associated issue OR is this a dependency/testcase change?

Reviewer Checklist

• Changes appear to address issue?
• Reviewer has tested changes by following the provided instructions?
• Changes appear not to be breaking changes?
• Appropriate automated tests included?
• Code style and in-line documentation are appropriate?

[codecov]

Codecov Report

Attention: Patch coverage is 81.69014% with 26 lines in your changes missing coverage. Please review.

Project coverage is 50.25%. Comparing base (d74e1b1) to head (39e66ca).

Files with missing lines	Patch %	Lines
src/selection/Selection.js	65.95%	16 Missing ⚠️
src/api/notifications/NotificationAPI.js	86.66%	6 Missing ⚠️
src/api/notifications/NotificationManager.js	91.83%	4 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (d74e1b1) and HEAD (39e66ca). Click for more details.

HEAD has 14 uploads less than BASE

Flag	BASE (d74e1b1)	HEAD (39e66ca)
unit	7	1
e2e-ci	8	0

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7910      +/-   ##
==========================================
- Coverage   57.67%   50.25%   -7.43%     
==========================================
  Files         678      679       +1     
  Lines       27428    27506      +78     
  Branches     2691     2691              
==========================================
- Hits        15820    13823    -1997     
- Misses      11270    13342    +2072     
- Partials      338      341       +3     

Flag	Coverage Δ		*Carryforward flag
e2e-ci	?
e2e-full	23.47% <ø> (-18.50%)	⬇️	Carriedforward from 894a5f3
unit	49.34% <81.69%> (-0.16%)	⬇️

*This pull request uses carry forward flags. Click here to find out more.

Files with missing lines	Coverage Δ
src/MCT.js	96.96% <100.00%> (+0.03%)	⬆️
src/api/notifications/NotificationManager.js	91.83% <91.83%> (ø)
src/api/notifications/NotificationAPI.js	92.15% <86.66%> (-6.69%)	⬇️
src/selection/Selection.js	65.06% <65.95%> (-26.49%)	⬇️

... and 209 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d74e1b1...39e66ca. Read the comment docs.