feat(ras-acc): add basic SVG handling?

[laurelfulford]

All Submissions:

• Have you followed the Newspack Contributing guideline?
• Does your code follow the WordPress' coding standards and VIP Go coding standards?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes proposed in this Pull Request:

I'm not sure we actually need this, but this PR takes a swing at consolidating all the SVGs used in the Newspack UI, and sets up a couple function to return or echo them.

This is a riff off of what's in the Newspack Theme, and might be overkill/clunky - it could also work to centralize and add on the individual functions that return the markup, like we're already doing in RAS-ACC -- open to feedback on that!

I haven't made any changes on the Newspack Blocks side yet; the duplication seems like an okay trade off for still having the SVGs work when the Newspack Plugin is disabled, but if this seems like a good plan we could do something similar there for the checkout-related SVGs.

How to test the changes in this Pull Request:

1. Apply the PR.
2. Check the ?ui-demo and make sure the SVGs are loading.
3. View as a non-logged in user and confirm the Sign In link in the header still has the icon.

Other information:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes, as applicable?
• Have you successfully ran tests with your changes locally?

[adekbadek]

Nice! I think it's a needed change, makes the code much more readable and abstracts away the clunkiness of SVG markup.

[laurelfulford]

Thanks @adekbadek! 🙌

[matticbot]

🎉 This PR is included in version 4.1.0-epic-ras-acc.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[matticbot]

🎉 This PR is included in version 4.2.0-epic-ras-acc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀