[Nautilus-list] Re: [PATCH] more desktop==home fixes
- From: Darin Adler <darin bentspoon com>
- To: David Emory Watson <dwatson cs ucr edu>
- Cc: nautilus-list lists eazel com, alexl redhat com
- Subject: [Nautilus-list] Re: [PATCH] more desktop==home fixes
- Date: Sun, 24 Mar 2002 08:18:37 -0800
> + && !eel_preferences_get_boolean
(NAUTILUS_PREFERENCES_DESKTOP_IS_HOME_DIR)) {
It seems wrong for nautilus_link_local_is_home_link to return false if
the preference is true. That should be handled at a higher level. It's
not like the home link becomes "not a home link" because of the
preferences setting. In particular, this makes it so the home link
doesn't get updated properly if the preference is off, which is not
necessarily a good idea. You already made a change at the higher level
to not re-create the home link if the preference is true.
It seems OK to have nautilus_link_local_is_special_link do this check,
although I'd prefer a more-specific name that tells you what's special
about these links (not deletable by the user, maybe?).
Otherwise, patch looks fine. Please commit.
-- Darin
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]