Bug 168946 - Properties dialog is modal for the current folder and not modal for icons
Summary: Properties dialog is modal for the current folder and not modal for icons
Status: RESOLVED FIXED
Alias: None
Product: dolphin
Classification: Applications
Component: general (show other bugs)
Version: 16.12.2
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Peter Penz
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-12 02:09 UTC by David Benjamin
Modified: 2019-11-28 15:26 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Benjamin 2008-08-12 02:09:04 UTC
Version:           1.1.80 (using Devel)
Installed from:    Compiled sources
OS:                Linux

If one right-clicks an icon in Dolphin and selects "Properties", the resulting dialog is not modal, but right-clicking a blank area in the folder results in a modal properties dialog.

The dialog should probably be non-modal for consistency since there isn't anything particular in the dialog that needs to be completed before returning.
Comment 1 Dario Andres 2008-08-12 03:08:01 UTC
Using KDE 4.1.1 (KDE 4.1.0 (4.1 >= 20080722)) (KDEmod) in ArchLinux i686:
I can confirm this behaviour. This should be marked as wish
Comment 2 Peter Penz 2008-08-12 07:48:01 UTC
Thanks for the pointer, I was not aware about this. I'll fix it ASAP...
Comment 3 Peter Penz 2008-08-12 20:47:06 UTC
SVN commit 845987 by ppenz:

Use a modeless properties dialog instead of a modal one, when opening it above the viewport.

BUG: 168946


 M  +6 -2      dolphincontextmenu.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=845987
Comment 4 Peter Penz 2008-08-12 20:47:34 UTC
SVN commit 845990 by ppenz:

Backport of 845987: Use a modeless properties dialog instead of a modal one, when opening it above the viewport.

CCBUG: 168946

 M  +6 -2      dolphincontextmenu.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=845990