Re: [PATCH] Add Pathbar to Nautilus Browser

[Jamie McCracken <jamiemcc blueyonder co uk>]

Alexander Larsson wrote:
> On Thu, 2005-07-07 at 20:03 +0100, Jamie McCracken wrote:
>
> > Alexander Larsson wrote:
> >
> >
> > > First a bit on the general UI. I don't want to have the pathbar in
> > > addition to the location bar, I want it to replace the location bar. For
> > > a long time a major goal of nautilus has been to avoid displaying
> > > pathnames (as in absolute pathnames or uris) in the UI. Going from
> > > location bar will help with this. It will also help us look less like a
> > > web browser, which is nice, as that can confuse people at times.
> > >
> > > Of course, we can't totally remove the location bar, as its needed for
> > > typing in addresses, and some people still might want it. What we'll do
> > > is put the path bar and the location bar in the same place, but only
> > > show one at a time. Normally we show the pathbar, but on "go ->
> > > Location" in the menu (or ctrl+l) we temporarily show the location bar,
> > > similarly to how this is handled when the location bar is not visible.
> >
> >
> > okay done.
>
>
> This wasn't really what I meant. I changed it to how i think it should
> be and commited.

must say I prefer the path bar to the left of the sidebar but at least 
its in - thanks for that.

> > > The gtk+ pathbar code you used isn't the latest version. Gtk+ HEAD has a
> > > different behaviour, primary wrt rerooting. It:
> > > * only re-roots to the desktop
> > > * allows you to scroll left when things are re-rooted
> > > We should update the code to this version.
> >
> > Done. Found a bug in size_allocate which prevented it from applying the 
> > scroll arrows when you resize properly. (added on line 425 of 
> > nautilus_pathbar.c).
>
>
> Did you file this bug in bugzilla, so the gtk+ version can be fixed?

yes I filed it with patch at :
http://bugzilla.gnome.org/show_bug.cgi?id=309795

I'm not using gtk 2.7 so I assume its a bug!

--
Mr Jamie McCracken
http://www.advogato.org/person/jamiemcc/