Fullscreen Popover Closing

[Blackbaud-TrevorBurch]

Resolves #1504

[Blackbaud-TrevorBurch]

@Blackbaud-ToddRoberts would you mind to give this a look? Thanks!

[Blackbaud-ToddRoberts]

Close button should be the same as it is on Modal. Looks a few px too large here.

[Blackbaud-TrevorBurch]

Hey @Blackbaud-ToddRoberts there is a small difference in this PR due to a styling on the modal's close button being lost when the icon components were pulled out. That is fixed in #1987 and that PR will bring the modal's close button back to having a font-size of 20px as it was prior to 2.23.0 when the mistake in the icon switchover brought it down to 15px.

[Blackbaud-AlexKingman]

From a code perspective, this LGTM!

[codecov-io]

Codecov Report

❗ No coverage uploaded for pull request base (master@952b694). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #1985   +/-   ##
=========================================
  Coverage          ?   99.94%           
=========================================
  Files             ?      423           
  Lines             ?     9298           
  Branches          ?     1391           
=========================================
  Hits              ?     9293           
  Misses            ?        5           
  Partials          ?        0

Impacted Files	Coverage Δ
src/modules/popover/popover.module.ts	100% <100%> (ø)

---

Continue to review full report at Codecov.

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