I was just using the same convention as publisher, where this button isn't commented or explained.  
I'll update the commit.


On Wednesday, 20 November 2013 at 15:48, Paul Hayes wrote:

> In app/views/layouts/application.html.erb:
> > @@ -21,12 +21,19 @@ > <header class="navbar navbar-inverse navbar-fixed-top" role="banner"> > <div class="navbar-inner"> > <div class="container"> > + > + <a class="btn btn-navbar" data-toggle="collapse" data-target=".nav-collapse"> > + <span class="icon-bar"></span> > + <span class="icon-bar"></span> > + <span class="icon-bar"></span> > + </a> > +  
> A comment should explain this block, preferably with a link to the bootstrap docs. At first glance it doesn't mean anything.
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/alphagov/transition/pull/80/files#r7794791).
>  
>  
>