=?ISO-8859-1?Q?Re:_[PATCH]_Bug_327691_=3F_URI-bar?= =?ISO-8859-1?Q?_location=09persists_after_deletion?=



 
>There is one behaviour issue.
>Say you go into /tmp/1/2/3/4, then click on the "2" button, and then "4"
>is deleted. With the patch as-is we're transported to "3". That
>shouldn't happen in this case. Only if we're currently in or under the
>deleted path.

Thanks.  I'll make sure the button for the active target is properly set to the active state.

>
>Also, the client used for the monitoring worry me a bit. Really, the
>NautilusFile+pathbar pair *should* be unique, but I'd just feel better
>if you used the ButtonData pointer as the client.
I'll fix that.

>
>Otherwise this looks good. Did you get your cvs account yet?
Not yet. Haven't heard back from them yet.

Thanks!

Gene



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]