Add a url type for navbar icons

[maximlt]

Partially addresses #603

[choldgraf]

This seems pretty good to me! One quick typo I found but it looks good in general. Just to be clear, the CSS and behavior is no different from the "local" use-case right? If that's the case it's probably not a big deal that we don't demo this in our theme documentation .

also I think you'll need to update one of the regression tests just FYI!

[maximlt]

Just to be clear, the CSS and behavior is no different from the "local" use-case right? If that's the case it's probably not a big deal that we don't demo this in our theme documentation .

You're right, no change in CSS and behavior. So I've merged the Local Image Icons and URL Icons (the section I added originally) into an Image Icons section. I think it's still worth documenting it.

[choldgraf]

I think this one looks good to go in my opinion. The failing tests were because the pre-commit hook had the wrong black dependency specified. I've updated that on master and if you merge master into this branch, I bet the tests will pass. Wanna give that a shot?

[maximlt]

Sure let's see if now they pass :) I've merged master.

[choldgraf]

I've made a slight modification to the docs to fix a failing test and to help separate out the url and local examples a bit more. I think this is ready to go!

[choldgraf]

many thanks for the improvement @maximlt !