Re: Re: Bug#201854: [patch] Bug #201854: galeon: doesn't remember last path in filerequester in <INPUT type="FILE"> select

Tommi Komulainen <[email protected]>
Newsgroups gmane.comp.web.galeon.devel
Message-ID <[email protected]>
On Thu, 2003-07-24 at 23:40, Mark Howard wrote:
> On Thu, 2003-07-24 at 09:16, John Lightsey wrote:
> > tags 201854 + patch
> > thanks
> > 
> > The attached patch creates a new gconf variable to track the last used 
> > directory for the filerequester dialog.  It also clears this gconf variable 
> > (and the last download directory variable) when Galeon is started.
> > 
> > This ties in a bit to bug #197683 but I'll have to put together another patch 
> > to get that one fixed properly.

It's leaking the string from GConf.

Hmm, do we really want to maintain separate directories in memory? 
Wouldn't it be simpler to keep just one directory and whenever the file
selector is needed, it would default to that directory?  I'd think it
would be least surprising.

Also, state information shouldn't be saved in GConf, but that's our
mistake, need to fix it some day.  (gnome-config API should be well
suited for the job.)


-- 
Tommi Komulainen                                 [email protected]
GPG 1024D/68388EE6    6FD6 DD79 EB38 BF6F 3533  09C0 04A8 9871 6838 8EE6
signature.asc (application/pgp-signature, 189 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2 (GNU/Linux)

iD8DBQA/ISnwBKiYcWg4juYRAuJoAJ9AcZXYXhPXw4Tyb8nllEOkAqCELACfSois
KfU8gw7QEaP011L6/j2YQNA=
=OGCZ
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.