Quoting message written on Sunday 2014-05-25 14:07:12 by Carlos R. Mafra:
> On Sun, 25 May 2014 at 14:22:22 +0200, Josip Deanovic wrote:
> > 0.9x also brought a blue panel (I believe it's called switchpanel)
> > which is optional and whether you use the switchpanel or not, shaded
> > windows are not included in the list of windows that are able to get
> > focus when cycling using ALT+TAB.
>
> I don't think this is true.
>
> I believe that Amadeusz correctly identified the cause. You said you
> have "SwitchPanelImages = None". Try without that to see that your
> issue goes away, I believe.
I have tested it again with the current next branch.
This is the current behavior:
Shaded window does get focus if:
- the option "SwitchPanelImages = None" is not used
- the option "CycleIgnoreMinimized = NO" is set (note that shaded
window is not minimized and should act more like normal window)
NOTE: - if covered by other windows, shaded window doesn't rise
although the CirculateRaise option is set to YES
- if selected by switchpanel, a shaded window will unshade
Shaded window does not get focus nor does it rise if:
- the option "SwitchPanelImages = None" is used (no switchpanel)
In my opinion, in both cases shaded window should be included in
the cycle window list together with other normal windows.
And unless switchpanel is used, shaded window shouldn't be unshaded
when brought on top and focused.
> which I think explains Amadeusz observation that you need to set
> "SwitchPanelImages = None".
I have "SwitchPanelImages = None" set all the time (unless I am
testing switchpanel).
> Your patch removes the line setting flags.mapped to 0, so the condition
>
> (wwin->flags.mapped || include_unmapped)
>
> becomes true and the window is added to the list. Alternatively, you can
> fix the issue by changing the include_unmapped variable.
>
> And it seems to me that your patch chose the wrong solution. I guess
> the patch should clean up the logic behind associating include_unmapped
> with wPreferences.swtileImage!=0.
I'll look into the suggested lines.
--
Josip Deanovic
--
To unsubscribe, send mail to [email protected].
