NavigationClose fixed

[c0b41]

#1909

[oliviertassinari]

@shaurya947 I'm not really sure we want to expose Icons from the index.js.

   Icons: {
     NavigationMenu: require('./svg-icons/navigation/menu'),
     NavigationChevronLeft: require('./svg-icons/navigation/chevron-left'),
     NavigationChevronRight: require('./svg-icons/navigation/chevron-right'),
   },

I would rather remove it. We have 100+ icons. Are we going to expose all of them?
I think that we shouldn't, hence for clarity may be better to not expose them at all from the index.

[shaurya947]

@oliviertassinari I agree. It might be a better idea to have an index.js for the SVG icons instead.

[oliviertassinari]

@shaurya947 Yeah, could be a good idea. And add them to the documentation. It would close this issue #1475