{"action":"created","issue":{"url":"https://api.github.com/repos/mate-desktop/caja/issues/423","labels_url":"https://api.github.com/repos/mate-desktop/caja/issues/423/labels{/name}","comments_url":"https://api.github.com/repos/mate-desktop/caja/issues/423/comments","events_url":"https://api.github.com/repos/mate-desktop/caja/issues/423/events","html_url":"https://github.com/mate-desktop/caja/issues/423","id":86201645,"number":423,"title":"[for discussion] drag-n-drop behavior in bookmarks section of sidebar: what to do?","user":{"login":"monsta","id":5138986,"avatar_url":"https://avatars.githubusercontent.com/u/5138986?v=3","gravatar_id":"","url":"https://api.github.com/users/monsta","html_url":"https://github.com/monsta","followers_url":"https://api.github.com/users/monsta/followers","following_url":"https://api.github.com/users/monsta/following{/other_user}","gists_url":"https://api.github.com/users/monsta/gists{/gist_id}","starred_url":"https://api.github.com/users/monsta/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/monsta/subscriptions","organizations_url":"https://api.github.com/users/monsta/orgs","repos_url":"https://api.github.com/users/monsta/repos","events_url":"https://api.github.com/users/monsta/events{/privacy}","received_events_url":"https://api.github.com/users/monsta/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-06-08T14:37:41Z","updated_at":"2015-06-12T08:05:11Z","closed_at":null,"body":"@stefano-k @flexiondotorg @NiceandGently @sunweaver @willysr @infirit \r\n\r\n\r\nSome history:\r\n\r\n1. Some time ago, drag-n-drop of files to the bookmarked folders didn't work (https://github.com/mate-desktop/caja/issues/345).\r\n2. Later it has been fixed by @NiceandGently in https://github.com/mate-desktop/caja/commit/910b9141ac634a86d8f53fda534e33787a160efb.\r\n3. I've filed [this Debian bug report](https://bugs.debian.org/786395), asking to make a patch for Debian out of that commit.\r\n4. During the patch testing, @sunweaver found out that adding new folders via drag-n-drop to bookmarks list does not work anymore.\r\n\r\nThe choice:\r\n\r\n- if we revert https://github.com/mate-desktop/caja/commit/910b9141ac634a86d8f53fda534e33787a160efb, we'll be able to add new folders to bookmarks via drag-n-drop. Like it's currently done in Caja 1.8: drag some folder to the empty space in bookmarks section, then a horizontal line appears, then you can drop the folder there, and it will become a new bookmark. But we'll also reintroduce https://github.com/mate-desktop/caja/issues/345 back.\r\n- if we leave it as is, we'll be able to drag-n-drop files into some existing bookmarked folder, but won't be able to add new folders to bookmarks via drag-n-drop.\r\n\r\nCurrent situation:\r\n\r\nThe patch isn't accepted in Debian (and Ubuntu), and we have Caja 1.8 there, so currently https://github.com/mate-desktop/caja/issues/345 affects these distros.\r\nOnce 1.10 makes it there, https://github.com/mate-desktop/caja/issues/345 will be fixed, but the ability to add new folders to bookmarks via drag-n-drop will be lost.\r\n\r\nUpstream situation:\r\n\r\n@NiceandGently found out that Nautilus now has both behaviors - that is, it allows both drag-n-drop of files into the existing bookmarks and adding new folders to bookmarks. It shows which action will be done so the user could distinguish between them and choose the one he wants.\r\nIt had been implemented somewhere between 3.8 and 3.10.\r\n\r\n**Bad news**: during the development of Nautilus 3.10, they made their own [GtkPlacesSidebar widget](https://developer.gnome.org/gtk3/stable/GtkPlacesSidebar.html) and switched Nautilus to it. So we don't have any upstream code to cherry-pick - we need to analyze GtkPlacesSidebar code and try to implement the same logic in Caja to fix our issue. This is not easily and quickly done...\r\n\r\nNemo, the Cinnamon file manager, also has both behaviors working, but again, the amount of changed code there is too big (see https://github.com/linuxmint/nemo/commit/f96357dcec0d4a7a90ce10d249ef164d9b093434 for example), and picking something we need from it is not a quick job.\r\n\r\nSo what should we do here?\r\nShould we consider the current 1.10 behavior a regression and revert it, or should we leave it as is and tell the users that it's a new feature? :laughing: \r\n\r\nPersonally, I'm inclined to do the latter.\r\nFirst, drag-n-drop of files to bookmarks should be fixed. If it doesn't work, there's no other quick way to do the same thing.\r\nSecond, Caja already has a quick way to add a new bookmark (via menu or Ctrl-D shortcut), so having an additional drag-n-drop action that does the same thing isn't *that* necessary."},"comment":{"url":"https://api.github.com/repos/mate-desktop/caja/issues/comments/111403092","html_url":"https://github.com/mate-desktop/caja/issues/423#issuecomment-111403092","issue_url":"https://api.github.com/repos/mate-desktop/caja/issues/423","id":111403092,"user":{"login":"sunweaver","id":407231,"avatar_url":"https://avatars.githubusercontent.com/u/407231?v=3","gravatar_id":"","url":"https://api.github.com/users/sunweaver","html_url":"https://github.com/sunweaver","followers_url":"https://api.github.com/users/sunweaver/followers","following_url":"https://api.github.com/users/sunweaver/following{/other_user}","gists_url":"https://api.github.com/users/sunweaver/gists{/gist_id}","starred_url":"https://api.github.com/users/sunweaver/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sunweaver/subscriptions","organizations_url":"https://api.github.com/users/sunweaver/orgs","repos_url":"https://api.github.com/users/sunweaver/repos","events_url":"https://api.github.com/users/sunweaver/events{/privacy}","received_events_url":"https://api.github.com/users/sunweaver/received_events","type":"User","site_admin":false},"created_at":"2015-06-12T08:05:11Z","updated_at":"2015-06-12T08:05:11Z","body":"For the record, from yesterday's chat on IRC:\r\n\r\n14:22 < sunweaver> I never tried to copy files into bookmarks folders since the beginning of GNOMEv2\r\n14:23 < sunweaver> but used drag'n'drop for bookmark folders regularly.\r\n14:23 < sunweaver> so for me that commit is zero-gain and 100%-functionality loss.\r\n14:23 < raveit65> sunweaver, as i said for me it's no prob to revert it, i can use the commit as patch for fedora, i can't introduce this regression for fedora again\r\n14:24 < sunweaver> but I will ship the change in 1.10 and wait for community feedback.\r\n14:24 < sunweaver> hmmm...\r\n14:24 < sunweaver> so in Fedora someone complained about the lack of being able to copy files? Interesting.\r\n14:25 < sunweaver> I only know about my usage pattern regarding bookmarks.\r\n14:25 < raveit65> sunweaver, i fixed a fedora bug with that commit in the past, so i can't revert it\r\n14:25 < sunweaver> So no clue if others will complain...\r\n14:25 < raveit65> but i can use a patch\r\n14:25 < raveit65> np\r\n14:27 < sunweaver> I can also unpatch that commit in Debian, but I,d rather not want to ship a Debian specific patch if it can be avoided.\r\n14:27 < sunweaver> no clue on how to proceed in upstream...\r\n\r\n... So I am happy to see the previous functionality of caja bookmarks drag'n'drop continued..."}}