[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #22 from Ahmad Samir --- >From the "Version Fixed In:" field in the bug report, the fix should be in Frameworks 5.76. -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #21 from j...@forestfield.org --- Thanks for working on this! I was a bit confused as to when I should expect to be able to see the change you filed; should the change be visible in any of the following software versions on the following architecture? Operating System: Debian GNU/Linux KDE Plasma Version: 5.19.5 KDE Frameworks Version: 5.74.0 Qt Version: 5.15.2 Kernel Version: 5.9.0-4-amd64 OS Type: 64-bit Processors: 8 × AMD FX(tm)-8150 Eight-Core Processor Memory: 31.3 GiB of RAM Graphics Processor: NV106 -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 Méven Car changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED|RESOLVED Latest Commit|https://invent.kde.org/fram |https://invent.kde.org/fram |eworks/kio/commit/5f84f12d6 |eworks/kio/commit/9cc4b4c76 |8ef33478b7b6f0052061c135051 |e7b7c837608a5c22853e673bf73 |04ab|876f --- Comment #20 from Méven Car --- Git commit 9cc4b4c76e7b7c837608a5c22853e673bf73876f by Méven Car. Committed on 31/10/2020 at 17:42. Pushed by meven into branch 'master'. In file dialogs, select an existing dir when trying to create it M +5-0src/filewidgets/kdiroperator.cpp M +7-0src/filewidgets/kdiroperator.h M +1-0src/filewidgets/kfilewidget.cpp M +18 -5src/filewidgets/knewfilemenu.cpp M +9-0src/filewidgets/knewfilemenu.h https://invent.kde.org/frameworks/kio/commit/9cc4b4c76e7b7c837608a5c22853e673bf73876f -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 Bug Janitor Service changed: What|Removed |Added Status|REOPENED|ASSIGNED --- Comment #19 from Bug Janitor Service --- A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kio/-/merge_requests/199 -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 Ahmad Samir changed: What|Removed |Added CC||fa...@kde.org, ||meve...@gmail.com --- Comment #18 from Ahmad Samir --- (In reply to Patrick Silva from comment #17) > Created attachment 132722 [details] > sceenshot taken on neon unstable > > My screenshot shows the result when I try to create a directory that already > exists. > That's a different, new, feature not directly related to the commit in comment#9. https://invent.kde.org/frameworks/kio/-/merge_requests/180 We may need to adapt that new behaviour to the kdiroperator use case... > Operating System: KDE neon Unstable Edition > KDE Plasma Version: 5.20.80 > KDE Frameworks Version: 5.76.0 > Qt Version: 5.15.0 -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #17 from Patrick Silva --- Created attachment 132722 --> https://bugs.kde.org/attachment.cgi?id=132722=edit sceenshot taken on neon unstable My screenshot shows the result when I try to create a directory that already exists. Operating System: KDE neon Unstable Edition KDE Plasma Version: 5.20.80 KDE Frameworks Version: 5.76.0 Qt Version: 5.15.0 -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #16 from Ahmad Samir --- I've tested again and it seems work here, I've asked on IRC if someone running neon can test too (hopefully we'll get some feedback). -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #15 from Patrick Silva --- (In reply to Ahmad Samir from comment #14) > Do you know the git commit that the kio package on neon is built from? (it > was committed ~ 6 days ago to kio). unfortunately I do not know. -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #14 from Ahmad Samir --- I tested again with recent-ish git snapshot, seems to work here. Do you know the git commit that the kio package on neon is built from? (it was committed ~ 6 days ago to kio). -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #13 from Patrick Silva --- (In reply to Ahmad Samir from comment #12) > You mean with qbittorrent? or with the open/save file that's used in e.g. > kate? with the open/save file dialog -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #12 from Ahmad Samir --- You mean with qbittorrent? or with the open/save file that's used in e.g. kate? -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 Patrick Silva changed: What|Removed |Added Resolution|FIXED |--- Status|RESOLVED|REOPENED --- Comment #11 from Patrick Silva --- This bug is still reproducible on neon unstable. Operating System: KDE neon Unstable Edition KDE Plasma Version: 5.20.80 KDE Frameworks Version: 5.76.0 Qt Version: 5.15.0 -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #10 from Ahmad Samir --- Actually this particular bug is fixed in plasma-integration, in kdirselectdialog. So it's fixed in master and I've just cherry-picked it to Plasma/5.20. The KIO part is for the regular file open dialog and knewfilemenu. Sorry about the confusion. -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 Nate Graham changed: What|Removed |Added Version Fixed In||5.76 -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 David Faure changed: What|Removed |Added Latest Commit||https://invent.kde.org/fram ||eworks/kio/commit/5f84f12d6 ||8ef33478b7b6f0052061c135051 ||04ab Status|CONFIRMED |RESOLVED Resolution|--- |FIXED --- Comment #9 from David Faure --- Git commit 5f84f12d68ef33478b7b6f0052061c13505104ab by David Faure, on behalf of Ahmad Samir. Committed on 10/10/2020 at 12:43. Pushed by dfaure into branch 'master'. In file dialogs, creating a dir that already exists should select it If the user tries to create a new directory in the open/save file dialog, and that directory already exists, select/highlight it in the view. M +3-0src/filewidgets/kdiroperator.cpp M +14 -1src/filewidgets/knewfilemenu.cpp M +10 -0src/filewidgets/knewfilemenu.h https://invent.kde.org/frameworks/kio/commit/5f84f12d68ef33478b7b6f0052061c13505104ab -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #8 from Ahmad Samir --- https://invent.kde.org/plasma/plasma-integration/-/merge_requests/8 -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 Patrick Silva changed: What|Removed |Added CC||bugsefor...@gmx.com -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #7 from Ahmad Samir --- I see; IINM, it's the same dialogue, from KIO KFileWidgets and co., but with a different option set (where you can only select dirs), I'll test that one too. -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082
--- Comment #6 from j...@forestfield.org ---
Thanks for the demonstration animation.
In reference to:
https://invent.kde.org/frameworks/kio/uploads/6df68f21d5ccca1ac7be6b2b21b40b1b/Peek_2020-09-19_16-41.gif
that in Dolphin, you've created a "kt" directory where this directory already
exists. You got an error ("A folder named
/home/ahmad/dev/kio/building-dir/bin/kt already exists") and the error panel
has an OK button. You clicked the OK button to dismiss that error panel, the
error panel disappeared, and then the animation loops.
I think there are a couple of misunderstandings in this discussion:
1. I'm talking about a different panel than what you're showing in the
animation: What you're showing is in Dolphin, but what I'm talking about is the
behavior of the open/save directory panel applications use when they want you
to point a program to a directory. In this panel, one does not see all of the
same icons and have the same functionality as a Dolphin file browser window.
I've uploaded a WebM attachment (Attachment 131791) to show the panel I'm
talking about.
2. The context is different: In the open/save directory panel one can create a
new folder (via the "New Folder..." button) and it is there that I think
creating a folder with the same path as one that already exists should not
return an error but instead take the user to that folder just as if the folder
were just created because in the context of this panel one is trying to say
something like "I wish to point the application to this specified folder" and
let the user get on with what they were working on.
To get to the panel I'm referring to, I used qBitTorrent version 4.2.5 (64-bit)
on Debian GNU/Linux:
Operating System: Debian GNU/Linux
KDE Plasma Version: 5.17.5
KDE Frameworks Version: 5.70.0
Qt Version: 5.14.2
Kernel Version: 5.8.0-1-amd64
OS Type: 64-bit
Processors: 8 × AMD FX(tm)-8150 Eight-Core Processor
Memory: 31.3 GiB of RAM
I imagine that there are other apps one could use to get to the same panel.
This panel is chiefly about manipulating directories -- one can make a new
directory (as you see I tried to do) but making a directory that matches a
directory which already exists produces an error instead of taking the user to
that directory.
I'm not well-versed in the set of KDE or Qt panels, so I'm not sure if we're
both referring to the same panel (perhaps with a different configuration) or
the same error. My main point is that in the middle of doing something else I
think that getting to the desired directory should trump the details of whether
that directory exists.
--
You are receiving this mail because:
You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #5 from j...@forestfield.org --- Created attachment 131791 --> https://bugs.kde.org/attachment.cgi?id=131791=edit The panel I'm referring to showing the error I think is undesirable. -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #4 from Ahmad Samir --- I've posted a gif to this merge request: https://invent.kde.org/frameworks/kio/-/merge_requests/141 -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 --- Comment #3 from j...@forestfield.org --- Thanks for the feedback. I'd like to know what "select the existing dir, then user then could press Enter to open it and save there" looks like and what this might work like before I would agree to that because given only this description a panel indicating an error sounds like more typing and clicking than I'd expect to want to do. I also don't think that indicating an extant directory is an error, no matter whether I indicate that directory by clicking or typing. Most of the time I indicate a directory to this panel, I need to work inside the directory that I indicated whether that directory has items in it or not. If I need a new empty directory, I would create the new empty directory I need and indicate that directory. -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 Ahmad Samir changed: What|Removed |Added CC||a.samir...@gmail.com --- Comment #2 from Ahmad Samir --- But what if the user actually wants to save the file in a new empty dir? How about a middle ground? we could display the error message and select the existing dir, then user then could press Enter to open it and save there. -- You are receiving this mail because: You are watching all bug changes.
[frameworks-kio] [Bug 424082] When trying to create a directory that already exists using the Open/save dialog, it should just should take user instead of showing an error
https://bugs.kde.org/show_bug.cgi?id=424082 Nate Graham changed: What|Removed |Added Summary|Open/save directory panel |When trying to create a |should take user to extant |directory that already |directory, not show error |exists using the Open/save |when trying to create an|dialog, it should just |directory that already |should take user instead of |exists |showing an error Status|REPORTED|CONFIRMED Severity|wishlist|minor CC||n...@kde.org Ever confirmed|0 |1 --- Comment #1 from Nate Graham --- Seems reasonable enough. -- You are receiving this mail because: You are watching all bug changes.
