Namespaced icons

[ubbe-xyz]

Namespaced icons

[ubbe-xyz]

@poteirard good point 👌🏻

We could do this as a first step?

Summarising pros and cons I can see from namepsacing:

Cons

• Need to remember structure to use them
  • I guess TS integrations will hint to autocomplete with should make this less painful ?
• <Icons /> by itself would render null I guess 😏

Pros

• fewer imports ( as per the example above )
• better readability ? ( <Icons.ZopaLogo /> vs <ZopaLogoIcon /> )

[poteirard]

fewer imports ( as per the example above)

That's not 100% true you can always do import * as RC from 'react-components'; and then <RC.ZopaLogo />

[thegrinder]

I agree with @lluia, it does make it clearer and easier to import

[poteirard]

What if we export the icons with
ProfileIcon instead then? I'm not a big fan of namespacing to be honest because you have to remember the structure/tree.

[ubbe-xyz]

Addressed on 2.0 ☝️