Gala

Merge lp://staging/~lyubodzhamov/gala/fix-1220179 into lp://staging/gala/freya

fix-1220179
Merge into gala-freya

Proposed by Lyubomir Dzhamov on 2016-05-20

Status:

Rejected

Rejected by:

Zisu Andrei on 2016-11-10

Proposed branch:

lp://staging/~lyubodzhamov/gala/fix-1220179

Merge into:

lp://staging/gala/freya

Diff against target:

12 lines (+3/-0)

1 file modified

src/WindowListener.vala (+3/-0)

To merge this branch:

bzr merge lp://staging/~lyubodzhamov/gala/fix-1220179

High

Confirmed

Link a bug report

Reviewer	Review Type	Date Requested	Status
Rico Tzschichholz		2016-05-20	Needs Fixing on 2016-05-21
Review via email: mp+295372@code.staging.launchpad.net

Revision history for this message

Rico Tzschichholz (ricotz) wrote on 2016-05-21:

This looks like solving the problem if an application starts up.
But if a running application sets its urgency flag it will be raised as well which is not wanted.

review: Needs Fixing

Revision history for this message

Lyubomir Dzhamov (lyubodzhamov) wrote on 2016-05-21:

> This looks like solving the problem if an application starts up.
> But if a running application sets its urgency flag it will be raised as well
> which is not wanted.

Hi there Rico,
I'm new to the technology and I don't have very deep understanding of the window management mechanism yet.
Why don't we want to raise applications with urgency flag set? It is not stealing the input focus. My concern was if it is going to show on top of let's say a full screen video playing, but it is not happening.

Revision history for this message

Zisu Andrei (matzipan) wrote on 2016-11-10:

Rejecting since solution is not what is desired.