Yeah I think the nav should be contained in a backbone view too. If you
want you could leave it as jquery for this pull request - I'll merge it in
and then rework it as backbone when I get time.
On 24 Oct 2013 13:23, "Richard" <notifications@github.com> wrote:

> In js/hal/views/browser.js:
>
> > @@ -16,5 +17,13 @@ HAL.Views.Browser = Backbone.View.extend({
> >
> >      this.$el.html(this.explorerView.el);
> >      this.$el.append(this.inspectorView.el);
> > -  },
> > +
> > +    var entryPoint = this.entryPoint;
> > +
> > +    $("#entryPointLink").click(function(event) {
>
> Nope - it is outside.
>
> In creation of new HAL.Browser: container: $('#browser')
>
> And in the HTML:
>   The HAL Browser
>
>    - Go To Entry Point <#141ea6b1c94e99ff_/>
>    - About The HAL Browser <https://github.com/mikekelly/hal-browser>
>
>      </div>
>   </div>
> </div>
>
> <div id="browser" class="container-fluid"></div>
>
> Would you like me to move the container element outside the navbar, or use
> JQuery events?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mikekelly/hal-browser/pull/25/files#r7186045>
> .
>