Thanks for writing a patch for this! A quick note: xfdesktop_move_all_cached_icons_to_desktop(XfdesktopIconView * { #ifdef ENABLE_FILE_ICONS GList *l, *leftovers = NULL; XfdesktopFileIconManager *fmanager = NULL;
TRACE("entering"); if(XFDESKTOP_IS_FILE_ICON_MANAGER(icon_view->priv->manager)) fmanager = XFDESKTOP_FILE_ICON_MANAGER(icon_view->priv- if(fmanager == NULL) return; + xfdesktop_file_icon_save(fmanager); If we add the save function there you won't need the if check. I don't forsee any issues with your patch, but I won't have time to fully test this patch until the weekend, I just wanted to provide some quick feedback. If you're ok with my proposed change, either you or I can turn the diff into a git formatted patch for inclusion into xfdesktop. http://docs.xfce.org/contribute/dev/git/start provides info and links if you've never worked with git before and want to. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1307251 Title: xfdesktop doesn't save the arrangement of desktop icons after a resolution change To manage notifications about this bug go to: https://bugs.launchpad.net/xfdesktop/+bug/1307251/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs