Ubuntu
unity8 package

[QtComp] Indicator does not close on invoking external application

Bug #1348686 reported by Omer Akram
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
unity8 (Ubuntu)
Fix Released
High
Unassigned

Bug Description

The top bar indicator menu does not close when you activate a setting app from it.

1. drag down sound icon from the top
2. tap 'Sound settings.'

What happens
the settings open behind the indicator as the indicator stays on top.

Tags: qtcomp

Related branches

lp:~unity-team/unity8/mirCompositor
Gerry Boland (community): Approve
PS Jenkins bot (community): Needs Fixing (continuous-integration)
Michael Zanetti (community): Approve (c++ code review)
Michael Terry: Approve (debian-packaging)
Vesa Rautiainen (community): Approve (design)
Diff: 6483 lines (+3413/-1522)
52 files modified
debian/changelog (+14/-0)
debian/control (+1/-2)
plugins/Utils/CMakeLists.txt (+1/-0)
plugins/Utils/easingcurve.cpp (+5/-1)
plugins/Utils/plugin.cpp (+2/-0)
plugins/Utils/windowkeysfilter.cpp (+57/-0)
plugins/Utils/windowkeysfilter.h (+51/-0)
qml/Components/Dialogs.qml (+217/-0)
qml/Components/DraggingArea.qml (+4/-0)
qml/Components/InputMethod.qml (+79/-0)
qml/Components/PageHeader.qml (+3/-1)
qml/Dash/Apps/RunningApplicationTile.qml (+45/-37)
qml/Dash/DashContent.qml (+4/-0)
qml/Launcher/Launcher.qml (+1/-1)
qml/Panel/Indicators.qml (+3/-0)
qml/Shell.qml (+171/-393)
qml/Stages/Animations/BaseSurfaceAnimation.qml (+93/-0)
qml/Stages/Animations/DarkenAndFadeInAnimation.qml (+48/-0)
qml/Stages/Animations/SwipeFromBottomAnimation.qml (+48/-0)
qml/Stages/Animations/SwipeUpAnimation.qml (+49/-0)
qml/Stages/PhoneStage.qml (+173/-265)
qml/Stages/SidestageHandle.qml (+0/-22)
qml/Stages/Splash.qml (+68/-0)
qml/Stages/SpreadDelegate.qml (+193/-29)
qml/Stages/StageWithSideStage.qml (+0/-413)
qml/Stages/SurfaceContainer.qml (+123/-0)
qml/Stages/SwitchingApplicationImage.qml (+0/-81)
qml/Stages/TabletStage.qml (+568/-0)
qml/Stages/TransformedSpreadDelegate.qml (+36/-30)
qml/Stages/TransformedTabletSpreadDelegate.qml (+369/-0)
src/CMakeLists.txt (+0/-3)
src/main.cpp (+14/-89)
tests/mocks/Unity/Application/Application.qmltypes (+0/-2)
tests/mocks/Unity/Application/ApplicationDBusAdaptor.cpp (+75/-0)
tests/mocks/Unity/Application/ApplicationDBusAdaptor.h (+43/-0)
tests/mocks/Unity/Application/ApplicationInfo.cpp (+71/-23)
tests/mocks/Unity/Application/ApplicationInfo.h (+22/-9)
tests/mocks/Unity/Application/ApplicationManager.cpp (+60/-31)
tests/mocks/Unity/Application/ApplicationManager.h (+15/-9)
tests/mocks/Unity/Application/CMakeLists.txt (+6/-2)
tests/mocks/Unity/Application/MirSurfaceItem.cpp (+172/-0)
tests/mocks/Unity/Application/MirSurfaceItem.h (+121/-0)
tests/mocks/Unity/Application/SurfaceManager.cpp (+73/-0)
tests/mocks/Unity/Application/SurfaceManager.h (+54/-0)
tests/mocks/Unity/Application/VirtualKeyboard.cpp (+55/-0)
tests/mocks/Unity/Application/VirtualKeyboard.h (+35/-0)
tests/mocks/Unity/Application/plugin.cpp (+16/-9)
tests/qmltests/Dash/Apps/tst_RunningApplicationsGrid.qml (+23/-2)
tests/qmltests/Launcher/tst_Launcher.qml (+11/-2)
tests/qmltests/Panel/tst_ActiveCallHint.qml (+1/-0)
tests/qmltests/Stages/tst_PhoneStage.qml (+9/-56)
tests/qmltests/tst_Shell.qml (+111/-10)
Revision history for this message
Gerry Boland (gerboland) wrote :

I can reproduce, but it appears system settings crashes on startup. Relevant part of the unity8 log:

qtmir.applications: ApplicationManager::onProcessStarting - appId= "ubuntu-system-settings"
qtmir.applications: ApplicationController::desktopFilePathForAppId - appId= "ubuntu-system-settings"
qtmir.applications: DesktopFileReader::DesktopFileReader - this= 0x2b95fd0 appId= "ubuntu-system-settings"
qtmir.applications: DesktopFileReader::loadDesktopFile - this= 0x2b95fd0 desktopFile= "/usr/share/applications/ubuntu-system-settings.desktop"
loaded desktop file with name='System Settings', comment='', icon='system-settings', exec='system-settings %u', path='', stagehint='SideStage'
qtmir.applications: Application::Application - appId= "ubuntu-system-settings" state= 0
qtmir.applications: Application::setStage - appId= "ubuntu-system-settings" stage= 0
qtmir.applications: ApplicationManager::add - appId= "ubuntu-system-settings"
qtmir.applications: ApplicationManager::focusApplication - appId= "ubuntu-system-settings"
qtmir.applications: ApplicationManager::resumeApplication - appId= "ubuntu-system-settings"
qtmir.applications: Application::setFocused - appId= "ubuntu-system-settings" focused= true
qtmir.applications: ApplicationManager::move - from= 0 to= 0
qtmir.applications: ApplicationManager::focusApplication - appId= "ubuntu-system-settings"
qtmir.applications: ApplicationManager::resumeApplication - appId= "ubuntu-system-settings"
qtmir.applications: ApplicationManager::focusApplication - appId= "ubuntu-system-settings"
qtmir.applications: ApplicationManager::resumeApplication - appId= "ubuntu-system-settings"

qtmir.applications: ApplicationManager::onProcessFailed - appId= "ubuntu-system-settings" duringStartup= true

qtmir.applications: ApplicationManager::remove - appId= "ubuntu-system-settings"
qtmir.applications: Application::~Application
qtmir.applications: DesktopFileReader::~DesktopFileReader
qtmir.applications: ApplicationManager::onProcessStopped - appId= "ubuntu-system-settings"
ApplicationManager::onProcessStopped reports stop of appId= "ubuntu-system-settings" which AppMan is not managing, ignoring the event

Revision history for this message
Gerry Boland (gerboland) wrote :

Yep, if app does not crash, the indicator menu still does not close.

Changed in unity8:
status: New → Confirmed
importance: Undecided → High
Gerry Boland (gerboland)
Changed in unity8:
status: Confirmed → Fix Committed
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu):
importance: Undecided → High
status: New → Fix Committed
Michał Sawicz (saviq)
no longer affects: unity8
Revision history for this message
kevin gunn (kgunn72) wrote :

fixed in general.
but there is this bug which requires a precondition
bug 1362017

Changed in unity8 (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.