Bug 305839

Summary:	Opening terminal from Album List opens Terminal in background
Product:	[Applications] digikam	Reporter:	Thomas Damgaard <thomasdn>
Component:	Portability-Interroperability	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	caulier.gilles, smit.meh
Priority:	NOR
Version:	2.5.0
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed In:	6.3.0
Sentry Crash Report:

Description Thomas Damgaard 2012-08-26 20:57:05 UTC

Opening terminal from Album List opens Terminal in background.

Reproducible: Always

Steps to Reproduce:
1. Right click an album
2. Open Terminal
3. No terminal is visible on the screen leaving the user confused.
Actual Results:  
Terminal is opened behind the Digikam window

Expected Results:  
Normally, when you open a program, it is opened in the foreground. This is also the default behavior when opening other programs from inside other programs in KDE. For example: opening dolphon from Konsole by right clicking and selecting Open Folder.

Comment 1 Smit Mehta 2012-08-27 09:26:28 UTC

Hi Thomas

On my system, ff you already have a terminal open, it will open it in background. Otherwise it will open a fresh terminal in the foreground. Is it the case with you? Or it opens in background only irrespective of anything?

Also, can you share your OS and KDE version?

Smit

Comment 2 caulier.gilles 2012-08-27 10:07:55 UTC

I can reproduce the problem on my system, with KDE 4.8.2

The way to open konsole is delegate to KDE of course.

Note that problem is not reproducible if you want to open in File Manager (Dolphin here)...

Code relevant is here :


Action from GUI :
https://projects.kde.org/projects/extragear/graphics/digikam/repository/revisions/master/entry/digikam/main/digikamapp.cpp#L803

Slot connected to open terminal :
https://projects.kde.org/projects/extragear/graphics/digikam/repository/revisions/master/entry/digikam/views/digikamview.cpp#L1073

Note : Open terminal use KToolInvocation::invokeTerminal() and Open In File MAnager use KRun(), both from KDE API

Gilles Caulier

Comment 3 Thomas Damgaard 2012-08-27 17:52:05 UTC

OS version: Kubuntu 12.04 LTS amd64
Platform Version 4.8.4 (4.8.4)

Comment 4 Jekyll Wu 2012-08-30 11:29:23 UTC


*** This bug has been marked as a duplicate of bug 221107 ***

Comment 5 caulier.gilles 2019-08-09 11:03:03 UTC

Fixed with bug #221107