Re: Re: Re: [RFC] Remove Xinerama related options

2012-02-18 Thread Martin Gräßlin
> > Anyway, no need to say that if there is new feature to play fullscreen
> > movies on several monitors, user should have the choice to enable or
> > disable this feature (I don't think all xinerama users want to watch
> > their movie across all their monitors ;-))
> 
> yes the script would have to be installed manually. But that's the general
> idea behind the scripting. Allowing users to do much much more with the
> window manager than what has been possible before.
Just for the record: https://git.reviewboard.kde.org/r/104014/

The script will be installed, the user just has to enable it. I think this is 
a quite nice solution to such problems.

Cheers
Martin

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Thomas Lübking

Am 20.01.2012, 19:08 Uhr, schrieb Shaun Reich :


lxr has both C++ understanding (which is that default textbox you see,

Thanks a lot.

It's the KGlobalSettings::splashScreenDesktopGeometry() in kdeglobals.cpp  
what sounds like legacy code(*) since nowadays splashscreens tend to be  
_NET_WM_WINDOW_TYPE_SPLASH, are managed and placed by kwin as  
placeOnMainWindow() what in doubt means "center of the main windows  
screen" - no idea whether KDE splashscreens actually perform a geometry  
request, but that sounds pointless to me and is probably left over from  
pre-netwm times?!  *shrug*


Cheers,
Thomas

(*) professional and non offending way to say "bit rot" or "cruft" ;-)
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Martin Gräßlin
On Friday 20 January 2012 20:04:12 Hans Chen wrote:
> While we're talking about confusing options in System Settings, I've always
> wondered why there are two Xinerama options in Window Behavior (Focus tab)
> - "Separate screen focus" and "Active screen follows mouse". In particular,
> how does the latter work together with "Show unmanaged windows on"?
Honest answer: I have no clue what these options are about.
>
> Thought it could be worth taking into consideration when cleaning up the
> multiple monitor options.
Yes that is needed.
>
> With best regards,
> Hans
>
> On Fri, Jan 20, 2012 at 19:08, Shaun Reich  wrote:
> > On Fri, Jan 20, 2012 at 11:32 AM, Thomas Lübking
> >
> >  wrote:
> > > a) cool - wasn't aware that there's lxr.kde.org (just typed that into
> >
> > the
> >
> > > browser just after reading ;-)
> > > b) "how"? - afaik lxr only handles definitions, not "use random strings
> >
> > in
> >
> > > some function" and while 'Unmanaged' leads to the kwin Unmanged and some
> > > other, '"Unmanaged"' leads nowhere ->
> >
> > lxr has both C++ understanding (which is that default textbox you see,
> > identifier searching), as well as a more hidden feature under 'general
> > search' wherein you can search arbitrary strings in arbitrary files,
> > arbitrarily (like grep) ;-)
> >
> > supports regexp as well
> >
> > that's the one i tend to need the most honestly. though you need to be
> > careful when searching strings, as remember some may be like
> > 'unmanaged wi&ndow', which would just screw your whole search over ;)
> >
> > --
> > Shaun Reich,
> > KDE Software Developer (kde.org)
> > ___
> > Plasma-devel mailing list
> > Plasma-devel@kde.org
> > https://mail.kde.org/mailman/listinfo/plasma-devel


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Hans Chen
While we're talking about confusing options in System Settings, I've always
wondered why there are two Xinerama options in Window Behavior (Focus tab)
- "Separate screen focus" and "Active screen follows mouse". In particular,
how does the latter work together with "Show unmanaged windows on"?

Thought it could be worth taking into consideration when cleaning up the
multiple monitor options.

With best regards,
Hans

On Fri, Jan 20, 2012 at 19:08, Shaun Reich  wrote:

> On Fri, Jan 20, 2012 at 11:32 AM, Thomas Lübking
>  wrote:
> > a) cool - wasn't aware that there's lxr.kde.org (just typed that into
> the
> > browser just after reading ;-)
> > b) "how"? - afaik lxr only handles definitions, not "use random strings
> in
> > some function" and while 'Unmanaged' leads to the kwin Unmanged and some
> > other, '"Unmanaged"' leads nowhere ->
>
> lxr has both C++ understanding (which is that default textbox you see,
> identifier searching), as well as a more hidden feature under 'general
> search' wherein you can search arbitrary strings in arbitrary files,
> arbitrarily (like grep) ;-)
>
> supports regexp as well
>
> that's the one i tend to need the most honestly. though you need to be
> careful when searching strings, as remember some may be like
> 'unmanaged wi&ndow', which would just screw your whole search over ;)
>
> --
> Shaun Reich,
> KDE Software Developer (kde.org)
> ___
> Plasma-devel mailing list
> Plasma-devel@kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Shaun Reich
On Fri, Jan 20, 2012 at 11:32 AM, Thomas Lübking
 wrote:
> a) cool - wasn't aware that there's lxr.kde.org (just typed that into the
> browser just after reading ;-)
> b) "how"? - afaik lxr only handles definitions, not "use random strings in
> some function" and while 'Unmanaged' leads to the kwin Unmanged and some
> other, '"Unmanaged"' leads nowhere ->

lxr has both C++ understanding (which is that default textbox you see,
identifier searching), as well as a more hidden feature under 'general
search' wherein you can search arbitrary strings in arbitrary files,
arbitrarily (like grep) ;-)

supports regexp as well

that's the one i tend to need the most honestly. though you need to be
careful when searching strings, as remember some may be like
'unmanaged wi&ndow', which would just screw your whole search over ;)

-- 
Shaun Reich,
KDE Software Developer (kde.org)
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Thomas Lübking

Am 20.01.2012, 17:11 Uhr, schrieb Martin Gräßlin :

So basically i guess this option is dead (unless it's read somewhere  
else

and exported to the root window - but i don't know. My root window looks
unspectacular in this regard)

lxr will help for that :-)
a) cool - wasn't aware that there's lxr.kde.org (just typed that into the  
browser just after reading ;-)
b) "how"? - afaik lxr only handles definitions, not "use random strings in  
some function" and while 'Unmanaged' leads to the kwin Unmanged and some  
other, '"Unmanaged"' leads nowhere ->


find -name "*.cpp" -exec grep -H '"Unmanaged"' '{}' \;

in kde-workspace finds the xinerama kcm and - likely unrelated ;-) -  
plasma/generic/dataengines/network/networkengine.cpp



Cheers,
Thomas
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Martin Gräßlin
On Friday 20 January 2012 16:52:00 Thomas =?ISO-8859-1?Q?L�bking?= wrote:
> Am 20.01.2012, 13:31 Uhr, schrieb Martin Gräßlin :
> > On Friday 20 January 2012 13:11:03 Alex Fiestas wrote:
> >> On Friday, January 20, 2012 11:32:13 AM Martin Gräßlin wrote:
> >> > From the userinterface the only useful option is "Show unmanaged
> >>
> >> windows
> >>
> >> > on" which should be kept.
>
> Errrheemmm ...  hehe  uu
>
> It writes "Unmanaged" to [Windows] in kdeglobals from where it is at least
> not read by KWin (which would not place unmanaged windows anyway)
> KWin reads XineramaPlacementScreen and ("somehow") uses it to place
> *managed* windows.
I always considered the option to be for "windows not yet managed" =>
"unmanaged".
> I however don't know where a GUI config for this would reside and there's
> no such entry in my entire ~/.kde/share/config dir - which contains
> vry old files...
Hmm that would explain why I had the feeling for quite some time that the
option is not working.
>
> So basically i guess this option is dead (unless it's read somewhere else
> and exported to the root window - but i don't know. My root window looks
> unspectacular in this regard)
lxr will help for that :-)
>
> On general topic, i'm pretty much with Sebastian - "Make one HUGE screen"
> ./. "Make many Screens" should be entirely sufficient*, having different
> placement, maximization and resistance behavior sounds weird and otherwise
> the kcm looks like it crosses krandr.
>
> Cheers,
> Thomas
>
> *not sure whether this would lower code complexity, though...


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Thomas Lübking

Am 20.01.2012, 13:31 Uhr, schrieb Martin Gräßlin :


On Friday 20 January 2012 13:11:03 Alex Fiestas wrote:

On Friday, January 20, 2012 11:32:13 AM Martin Gräßlin wrote:
> From the userinterface the only useful option is "Show unmanaged  
windows

> on" which should be kept.


Errrheemmm ...  hehe  uu

It writes "Unmanaged" to [Windows] in kdeglobals from where it is at least  
not read by KWin (which would not place unmanaged windows anyway)
KWin reads XineramaPlacementScreen and ("somehow") uses it to place  
*managed* windows.
I however don't know where a GUI config for this would reside and there's  
no such entry in my entire ~/.kde/share/config dir - which contains  
vry old files...


So basically i guess this option is dead (unless it's read somewhere else  
and exported to the root window - but i don't know. My root window looks  
unspectacular in this regard)


On general topic, i'm pretty much with Sebastian - "Make one HUGE screen"  
./. "Make many Screens" should be entirely sufficient*, having different  
placement, maximization and resistance behavior sounds weird and otherwise  
the kcm looks like it crosses krandr.


Cheers,
Thomas

*not sure whether this would lower code complexity, though...
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Martin Gräßlin
On Friday 20 January 2012 14:13:17 XI wrote:
> > so you would only need one of the options, the fullscreen one? And in
> > fact you only want that for vlc, right? You don't want the option for
> > let's say Firefox?
> > 
> > So e.g. a KWin script taking care of setting the fullscreen video player
> > to all screens would suit you better, right?
> 
> You are right, the only option I used is the fullscreen one. I used it
> with xine, but the same apply for vlc,  Could also be useful for
> picture viewer, but you are right, I wouldn't want it for other
> applications.
> 
> I don't know what's your idea behind the "script taking care of setting
> the fullscreen" ? Do you want to use "Special Window Settings..." menu
> for movie players ? If it works for setting a fullscreen spawning all
> monitors, that sounds a reasonable option for me.
> Or maybe you are thinking about something else ?
I'm talking about the scripting interface for KWin. I just wrote a script for 
that:

workspace.clientFullScreenSet.connect(function(client, set) {
  if (client.resourceName == "vlc" && set) {
client.geometry = {x: 0, y: 0, width: 1280+1920, height: 1024};
  }
});

works like a charm. Vlc spans both screens when going to fullscreen. For 4.9 
GHNS support for KWin scripts is scheduled. For more information about KWin 
scripting see [1] (Be aware that the API changes with 4.9)
> 
> Anyway, no need to say that if there is new feature to play fullscreen
> movies on several monitors, user should have the choice to enable or
> disable this feature (I don't think all xinerama users want to watch
> their movie across all their monitors ;-))
yes the script would have to be installed manually. But that's the general 
idea behind the scripting. Allowing users to do much much more with the window 
manager than what has been possible before.

Cheers
Martin

[1] http://techbase.kde.org/Development/Tutorials/KWin/Scripting


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Martin Gräßlin
On Friday 20 January 2012 13:31:49 todd rme wrote:
> On Fri, Jan 20, 2012 at 11:53 AM, Alex Fiestas  wrote:
> > The only reason I can think of is to create "video walls", meaning
> > multiple
> > monitors used as if they were 1.
> > 
> > Not sure if that can be done lower in the stack using xrandr for example.
> 
> But is that something that should be in this config?  I would think
> that would be something done on a window-by-window basis, rather than
> a system-wide setting.  I see two possible ways to handle this (not
> mutually exclusive):
> 
> 1. A window rule to ignore multi-monitor information when maximizing.
> 2. An option when right-clicking on the title bar to maximum ignoring
> multi-monitor information.
This would add even more complexity and would further clutter the window menu 
with rather useless options.

For such specific cases we have KWin scripts.

Cheers
Martin

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread todd rme
On Fri, Jan 20, 2012 at 1:31 PM, Martin Gräßlin  wrote:
> On Friday 20 January 2012 13:11:03 Alex Fiestas wrote:
>> On Friday, January 20, 2012 11:32:13 AM Martin Gräßlin wrote:
>> > From the userinterface the only useful option is "Show unmanaged windows
>> > on" which should be kept.
>>
>> Well imho as a heavy multi* everything user I don't see why a user would
>> want to force "Unmanaged windows" in one screen since usually your
>> attention is where your pointer is.
> there might be use cases for the other option. Think of a TV attached to the
> screen. You would never want windows to open there. On the other hand the
> mouse cursor won't be there either. So hmm maybe you're right ;-)
>
> Cheers
> Martin

Why would you assume your mouse isn't there?  That would be true if
you launch something using the application launcher, but if you use
krunner to launch stuff you could easily have the mouse on, say, a TV
or projector when you launch an application.

-Todd
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Martin Gräßlin
On Friday 20 January 2012 13:11:03 Alex Fiestas wrote:
> On Friday, January 20, 2012 11:32:13 AM Martin Gräßlin wrote:
> > From the userinterface the only useful option is "Show unmanaged windows
> > on" which should be kept.
>
> Well imho as a heavy multi* everything user I don't see why a user would
> want to force "Unmanaged windows" in one screen since usually your
> attention is where your pointer is.
there might be use cases for the other option. Think of a TV attached to the
screen. You would never want windows to open there. On the other hand the
mouse cursor won't be there either. So hmm maybe you're right ;-)

Cheers
Martin

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Martin Gräßlin
On Friday 20 January 2012 13:09:18 Xavier IZARD wrote:
> Martin Gräßlin a écrit :
> > Hi workspace developers,
> > 
> > KWin provides the option to turn off multi screen aware window management.
> > This results in windows being maximized over both screens or fullscreen
> > windows being stretched over all screens. These options make the code much
> > more complex and add confusing configuration dialogs to our workspace (see
> > attached screenshot).
> > 
> > Example code:
> > int Workspace::numScreens() const
> > {
> > 
> >  if (!options->xineramaEnabled)
> >  
> >  return 1;
> >  
> >  return Kephal::ScreenUtils::numScreens();
> > 
> > }
> > 
> > I personally fail to understand why the options are needed and why they
> > have been added in the first place. With git blame I was not able to go
> > back so far in the history to find when they were added. Note: the
> > xinerama code used also to be ifdefed, so I could imagine it being from a
> > time back when multi screens was a new feature.
> > 
> >> From the userinterface the only useful option is "Show unmanaged windows
> >> on"> 
> > which should be kept. All other options make in my opinion just no sense.
> > 
> > If nobody sees any good reason to keep these options I would prepare the
> > removal.
> > 
> > The reason is btw not to remove options, but to decrease the complexity of
> > the code as shown above and the fact that it is a break my system setting
> > which happened more than once that users complained about maximize not
> > working any more.
> > 
> > Cheers
> > Martin
> 
> Hi,
> 
> 
> Could be more clear about the behaviour of kwin without these options ?
> 
> I suppose that the behaviour would be the same as when all these options
> were checked ? (ie, maximize & fullscreen work on each screen
> independently, magnetic borders, ...)
yes, everything would be like the options are checked
> 
> What about persons who group monitors to do a huge monitor panel ?
> (video wall).
> What about people who use very large monitor needing dual video inputs ?
those can disable xinerama support through xorg.conf
> 
> 
> My personal usage is (was) this one : I uncheck the option "Enable
> multiple monitor window fullscreen support" (all other options are
> checked). So that :
> -> When working on my computer, the maximized windows spawn only one
> monitor at a time, I get magnetic borders, ...
> -> When watching movie (fullscreen mode), the display spawns all my
> monitors thus doing a video wall.
so you would only need one of the options, the fullscreen one? And in fact you 
only want that for vlc, right? You don't want the option for let's say 
Firefox?

So e.g. a KWin script taking care of setting the fullscreen video player to 
all screens would suit you better, right?
> 
> 
> Sorry, but what I see here is a powerful feature of KDE that is going to
> disappear ...
> 
> 
> Regards,
> Xavier
> 
> P.S. I am the "author" of the option "Enable multiple monitor window
> fullscreen support"
Thanks for providing your feedback. This is appreciated and quite helpfull.

Cheers
Martin

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Martin Gräßlin
On Friday 20 January 2012 11:53:22 Alex Fiestas wrote:
> The only reason I can think of is to create "video walls", meaning multiple
> monitors used as if they were 1.
for that you do not even need these options. You can resize your window as big 
as you want. Quite independent from the options.

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Re: [RFC] Remove Xinerama related options

2012-01-20 Thread Alex Fiestas
On Friday, January 20, 2012 12:37:12 PM Sebastian Kügler wrote:
> Just disabling Xinerama would be enough to achieve that.

Thought xinerama was something deprecated and that should not be used :/

I don't know where xinerama is exactly, is it at the same level of XRandR ? 
one below? one above?

Can I have XRandR without Xinerama? If so, can I achieve the "video wall" with 
it?

Anyway, videowall is a corner case.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel