Add a twig function called icon instead of using imports / macros.

@phiamo, don't merge this yet. 

This was just to illustrate one of the ideas I had about solving this. It works pretty well, in configuration you'd just do:

    mopa_bootstrap:
        icons:
            icon_set: fontawesome  # defaults to glyphicons so no BC here

This will allow us to add more icon styles and using a template instead of just a prefix will allow other fonts that may have data attributes or something. 

One issue I can think of is name collisions. Maybe instead we should rename the icon function to `mopa_bootstrap_icon`? It's kind of long but would fit with the other function namings like the navbar.. It would be nice to have it be configurable, so it would default to `icon` but you could change it if it caused problems, and have an alias  `mopa_bootstrap_icon` to use in the templates from the bundle. Not sure. 

Also note that I removed the `<span>` from the menu.html.twig and there should now only be a space after an icon. Otherwise there will be no space and it should be inline anyway.