feat/930-extend-toNewsKitIcon-types-2

[KayKostadinov]

closes #930

What

1. Background - why this is needed
2. What did you do
3. What does the reviewers should expect

I have done:

• Written unit tests against changes
• Written functional tests against the component and/or NewsKit site
• Updated relevant documentation

I have tested manually:

• The feature's functionality is working as expected on Chrome, Firefox, Safari and Edge
• The screen reader reads and flows through the elements as expected.
• There are no new errors in the browser console coming from this PR.
• When visual test is not added, it renders correctly on different browsers and mobile viewports (Safari, Firefox, small mobile viewport, tablet)
• The Playground feature is working as expected

Before:

After:

Who should review this PR:

How to test:

[KayKostadinov]

Theming e2e test fails when trying to release to prod due to a duplicate ID.

the failing pipeline:
https://app.circleci.com/pipelines/github/newscorp-ghfb/NewsKit/7493/workflows/daff3af7-9ed3-4ac0-89d3-bdefec61725a/jobs/168243
not reproducible locally

Disabling the test to unblock the pipeline

[pp-serviceaccount]

You can preview these changes on:
Documentation Site: http://ncu-newskit-docs-pr.s3-website-eu-west-1.amazonaws.com/930-extend-tonewskiticon-types-2
Storybook: http://ncu-newskit-docs-pr.s3-website-eu-west-1.amazonaws.com/930-extend-tonewskiticon-types-2/storybook