As part of this work, we should also update Dashboard to reference icons via the npm package rather than by a GitHub reference

It's also worth mentioning that something already exists in NPM
https://www.npmjs.com/package/rancher-icons

@nwmac the idea is to have separated implementations of this, as you cannot actually add the library to rancher/dashboard till you merge the changes in rancher/icons. The aim is to be able to do both in separated instances.
It would not be possible to do an actual review in that case.

What would you suggest otherwise as process for the review?

That's an old package - I think we'd publish as @rancher/icons - I don't think we have anyone who has access to that package.

We can have 2 PRs - 1 for the changes to publish, 1 for dashboard changes - both linked to fix this issue. The icons PR can be reviewed first, the icon package reviewed and then the dashboard PR reviewed.

Deal.