Is the Button, an  element with svg code and hidden link content inside, ARIA conform and well laid out?
SVG has a desc and a title tag that you're meant to use to describe the SVG:

&lt;a href="#menu" class="head__toggle" title="Toggle Menu"&gt;
    &lt;svg viewBox="0 0 70.115 53.162" class="head__btn"&gt;
        &lt;desc&gt;Toggle Menu&lt;/desc&gt;
        &lt;use xlink:href="#skmenu"&gt;&lt;/use&gt;
    &lt;/svg&gt;
&lt;/a&gt;


When you run it through CodeSniffer, you get this notice instead:


  Notice: Check that text of the link describes the purpose of the link.


That said, I don't buy that inline SVGs are the way to go, as it increases the size of the document and cannot be cached in a meaningful way.  I mean, it's not considered a good practice to place the entire contents of your CSS file in the head tag to reduce HTTP requests, why would it be good for SVGs?

You can still get most of the benefits Mr. Coyier talks about in his SVG For Everybody talk by using an external SVG embedded via the img tag, in addition to being able to cache it.


Take advantage of CSS for styling purposes (including media queries)
SVG sprite stack technique, meaning you won't have to set the viewBox information.