[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2018-03-27 Thread Maik Qualmann
https://bugs.kde.org/show_bug.cgi?id=385280

--- Comment #10 from Maik Qualmann  ---
The same for the showFoto setup. :-)

Maik

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2018-03-27 Thread acidrums4
https://bugs.kde.org/show_bug.cgi?id=385280

--- Comment #9 from acidrums4  ---
(In reply to Maik Qualmann from comment #8)
> Did you activate the option for the native file dialog in the digiKam
> Setup-> Miscellaneous?

Sweet mother of Qt, I'm so embarrased. I didn't noticed that. Thank you so much
:)

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2018-03-27 Thread Maik Qualmann
https://bugs.kde.org/show_bug.cgi?id=385280

--- Comment #8 from Maik Qualmann  ---
Did you activate the option for the native file dialog in the digiKam Setup->
Miscellaneous?

Maik

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2018-03-27 Thread acidrums4
https://bugs.kde.org/show_bug.cgi?id=385280

acidrums4  changed:

   What|Removed |Added

 CC||acidrums4+st...@gmail.com

--- Comment #7 from acidrums4  ---
Hmmm... At least for me with Digikam 5.9 (updated yesterday), both Digikam (at
least when selecting a location for a local collection) and Showfoto (on
opening/saving images) are using the default Qt selection dialog rather than
Plasma one... This is under Gentoo.

I'm not switching this bug to "reopened" (it would be rude from my part) but
hope you take a closer look on this. I really hoped this was fixed on 5.8, but
as 5.9 arrived and it's still the same...

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2017-12-25 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=385280

caulier.gil...@gmail.com changed:

   What|Removed |Added

  Component|general |Usability-Ergonomy

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2017-10-03 Thread Maik Qualmann
https://bugs.kde.org/show_bug.cgi?id=385280

Maik Qualmann  changed:

   What|Removed |Added

   Version Fixed In||5.8.0
  Latest Commit||https://commits.kde.org/dig
   ||ikam/0467f7c6e2ab191468d8d2
   ||5061c6afc9b2d32134
 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED

--- Comment #6 from Maik Qualmann  ---
Git commit 0467f7c6e2ab191468d8d25061c6afc9b2d32134 by Maik Qualmann.
Committed on 03/10/2017 at 08:09.
Pushed by mqualmann into branch 'master'.

use of native file dialogs is now configurable
FIXED-IN: 5.8.0

M  +2-3NEWS
M  +2-0libs/settings/applicationsettings.cpp
M  +3-0libs/settings/applicationsettings.h
M  +10   -0libs/settings/applicationsettings_miscs.cpp
M  +3-0libs/settings/applicationsettings_p.cpp
M  +6-3libs/settings/applicationsettings_p.h
M  +39   -12   libs/widgets/files/dfiledialog.cpp
M  +4-0libs/widgets/files/dfiledialog.h
M  +29   -13   showfoto/main/showfotosettings.cpp
M  +6-3showfoto/main/showfotosettings.h
M  +13   -7showfoto/setup/showfotosetupmisc.cpp
M  +6-0utilities/setup/setupmisc.cpp

https://commits.kde.org/digikam/0467f7c6e2ab191468d8d25061c6afc9b2d32134

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2017-10-02 Thread Maik Qualmann
https://bugs.kde.org/show_bug.cgi?id=385280

--- Comment #5 from Maik Qualmann  ---
Luca,

it crashes into Gtk. Yes Gilles, You're right I'm making an option in
setup/Miscs.

Maik

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2017-10-02 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=385280

caulier.gil...@gmail.com changed:

   What|Removed |Added

 CC||caulier.gil...@gmail.com

--- Comment #4 from caulier.gil...@gmail.com ---
Hi Maik,

Why not an option in setup/Miscs instead a compilation option ? After all it's
a runtime problem...

Gilles

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2017-10-02 Thread Luca Beltrame
https://bugs.kde.org/show_bug.cgi?id=385280

--- Comment #3 from Luca Beltrame  ---
(In reply to Maik Qualmann from comment #1)
> Yes, we do not use the native file dialogs, because it is e.g. under Ubuntu
> leads to a crash. However, I have meant that plasma users want native file

Is the crash in Qt, or in KDE software? Has a bug been reported? If it's in our
software (KDE) we should *fix* it instead of working it around.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2017-10-02 Thread Fabian Vogt
https://bugs.kde.org/show_bug.cgi?id=385280

--- Comment #2 from Fabian Vogt  ---
(In reply to Maik Qualmann from comment #1)
> Yes, we do not use the native file dialogs, because it is e.g. under Ubuntu
> leads to a crash. However, I have meant that plasma users want native file
> dialogs. Gilles, therefore my suggestion, we make it as a compile option
> activatable. Patch is already in process.

A compile option doesn't make much sense IMO, it needs to do runtime detection.
openSUSE for example offers a multitude of desktops you can run digikam on.
You can probably query whether a metaobject for KFileDialog is registered...

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2017-10-02 Thread Maik Qualmann
https://bugs.kde.org/show_bug.cgi?id=385280

Maik Qualmann  changed:

   What|Removed |Added

 CC||metzping...@gmail.com

--- Comment #1 from Maik Qualmann  ---
Yes, we do not use the native file dialogs, because it is e.g. under Ubuntu
leads to a crash. However, I have meant that plasma users want native file
dialogs. Gilles, therefore my suggestion, we make it as a compile option
activatable. Patch is already in process.

Maik

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2017-10-01 Thread Fabian Vogt
https://bugs.kde.org/show_bug.cgi?id=385280

Fabian Vogt  changed:

   What|Removed |Added

 CC||fab...@ritter-vogt.de

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385280] digikam does not use the native Plasma file dialog under Plasma

2017-10-01 Thread Luca Beltrame
https://bugs.kde.org/show_bug.cgi?id=385280

Luca Beltrame  changed:

   What|Removed |Added

Summary|digikam does not use the|digikam does not use the
   |native Plasma dialog under  |native Plasma file dialog
   |Plasma  |under Plasma

-- 
You are receiving this mail because:
You are watching all bug changes.