[digikam] [Bug 374248] New: importing photo, bad file selector

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

            Bug ID: 374248
           Summary: importing photo, bad file selector
           Product: digikam
           Version: 5.4.0
          Platform: Other
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: Import
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

Created attachment 103039
  --> https://bugs.kde.org/attachment.cgi?id=103039&action=edit
digikam file selector

if i import photos or folder
then
i get a file selector which is not dolphin and which is not populated with user
or system shortcuts
then
i can't reach:
- easily any folder
- my smartphone connected via wifi with kdeconnect or via bluetooth with
bluedevil

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

--- Comment #1 from Philippe ROUBACH <[hidden email]> ---
Created attachment 103040
  --> https://bugs.kde.org/attachment.cgi?id=103040&action=edit
dolphin file selector

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

[hidden email] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
         Resolution|---                         |INVALID
             Status|UNCONFIRMED                 |RESOLVED

--- Comment #2 from [hidden email] ---
digiKam use Qt5 dialog to be compatible everywhere. KDE dialog is deprecated
since version 5


Gilles Caulier

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

--- Comment #3 from Philippe ROUBACH <[hidden email]> ---
ok for a qt file selector
but
you don't answer about populating it with user and system shortcut
and in particular
bluedevil or kdeconnect shortcut

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

--- Comment #4 from [hidden email] ---
As i know, Qt5 file dialog do not permit this yet.

If you want this feature report as upstream to Qt bugzilla.

Note : the screenshot that you provide is a dolphin main view, not a file
dialog.

Gilles Caulier

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

--- Comment #5 from Philippe ROUBACH <[hidden email]> ---
Created attachment 103042
  --> https://bugs.kde.org/attachment.cgi?id=103042&action=edit
true dolphin file_selector

treu dolphin file selector

you cab see it is the same thing about shortcuts

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

Philippe ROUBACH <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|INVALID                     |---
             Status|RESOLVED                    |UNCONFIRMED

--- Comment #6 from Philippe ROUBACH <[hidden email]> ---
qt answer
here https://bugreports.qt.io/browse/QTBUG-57828

"This is not a Qt problem at all. The KDE theme plugin didn't get loaded."

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

--- Comment #7 from [hidden email] ---
I'm not wrong. I check GwenView code. Open/File give a similar dialog that you
attach in this report. Gwenview code use KFileDialog, which is relevant of
KDELibsSupport.

digiKam is a pure Qt5/KF5 application. We drop all KDE4 deprecated API. We will
not go back. Never.

If Qt5::QFileDialog support native theme, why not. But here this do not work.

In digiKam we have QFileDialog option DontUseNativeDialog turned on, which must
be disabled in this case. I will patch source code in this way. But as i said,
this do not change anything here. Perhaps something need to be installed in
Plasma desktop to take effect. I don't know.

Gilles Caulier

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

--- Comment #8 from [hidden email] ---
Git commit 0a5615d18d5e697e19e5c90fea1e3e4987e2db2c by Gilles Caulier.
Committed on 28/12/2016 at 14:20.
Pushed by cgilles into branch 'master'.

permit to use native file dialog

M  +0    -1    app/main/digikamapp.cpp
M  +0    -1    libs/dialogs/imagedialog.cpp
M  +7    -4    libs/widgets/common/dwidgetutils.cpp
M  +1    -1    utilities/assistants/firstrun/collectionpage.cpp
M  +0    -1    utilities/imageeditor/editor/editorwindow.cpp

https://commits.kde.org/digikam/0a5615d18d5e697e19e5c90fea1e3e4987e2db2c

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

--- Comment #9 from Philippe ROUBACH <[hidden email]> ---
please if you have technical argument please discuss with qt team.
i suppplied the address of the qt answer. there is the address of the person
who answered.

don't play ping pong with me.

i am not a techie. i am only a user who want an app easy to use.

thanks

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

Maik Qualmann <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]

--- Comment #10 from Maik Qualmann <[hidden email]> ---
Gilles,

The native file dialog works here under openSUSE. I see the KDE file dialog.
But see Bug 373820. If the native file dialog under Debian problem makes, we
get a problem with digiKam-5.4.0 release. My suggestion, we use for
digiKam-5.4.0 the pure Qt file dialog. And look for a way to recognize the
running desktop (possible with Qt >= 5.7.0) and use only the native file dialog
if a Plasma desktop is present.

Maik

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

--- Comment #11 from [hidden email] ---
Hi Maik, and Happy new year...

I don't know if Plasma desktop detection was possible with Qt5.7. This work
around is fine for me.

Gilles

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

--- Comment #12 from Maik Qualmann <[hidden email]> ---
Gilles,

Thanks, Happy New Year to you. You're right there is no way with Qt-5.7.0 to
recognize the desktop. I find another way.

Maik

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

--- Comment #13 from [hidden email] ---
Maik,

Dbus can be a solution. Not perfect so far, but look here :

https://cgit.kde.org/digikam.git/tree/libs/notificationmanager/dnotificationwrapper.cpp#n87

Take a care : Dbus is optional in digiKam.

Gilles

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

--- Comment #14 from Philippe ROUBACH <[hidden email]> ---
there is no more problem in my case with digikam 5.4.0

--
You are receiving this mail because:
You are the assignee for the bug.

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=374248

[hidden email] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
   Version Fixed In|                            |5.5.0

--
You are receiving this mail because:
You are the assignee for the bug.