Re: Issue 339 in xmonad: Support for toggling totem movie player fullscreen

Updates: Summary: Support for toggling totem movie player fullscreen Status: Accepted Labels: -Priority-Low Priority-Medium Comment #5 on issue 339 by wirtwo...@gmail.com: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 Nice work audunskaugen. Thanks for researching this and providing the _NET_WM_STATE_FULLSCREEN event hook (totemFullscreen.hs sample xmonad.hs). IMO, this will be good to get into contrib soon to make it available for more usage, testing and further development. There are some aspects to improve so if anyone wants to take on ownership and submit a patch addressing them that would be great. Otherwise if no action in a week or so I'll send a patch to add it to EwmhDesktops as is, initially as a standalone 'fullscreenEventHook'. Eventually, though, I think it would be good to include an improved version in the standard ewmhDesktopsEventHook, plus having it available by itself. (sjanssen feedback on IRC when asked for thoughts as to where to put this: "enabling the full screen event hook in EWMH would be fine, and would probably make sense.") As is it gives more function for totem users and surely some other similar apps too than the current (0.9) situation, but also with some additional work it seems like it can cover uses of the doFullFloat manageHooks with isFullscreen and the fullscreen query alexey attached to http://code.google.com/p/xmonad/issues/detail?id=333 It seems it covers tiled to fullscreen and back usage plus allowing multihead users to focus other screens without affecting fullscreened apps. In my quick tests it works for totem, flash, mplayer, mplayer-plugin, gnome-mplayer plugin. (This is with all fullscreen manageHooks disabled.) It doesn't seem to help with ooimpress unfortunately, nor evince. Areas to address: * Toggling seems less smooth than it should be. In my tests it appears that the window untiles, goes to hinted float size, then finally fullscreen although it's hard to tell as it's very rapid. There is some glitch anyway that I didn't research yet as to cause. Also other tiled windows from the same workspace look briefly as if they are composited with the fullscreen app since they change size to fill the space as the fullscreen app floats. Not sure how much improvement can practically be made here, but hopefully can something close to the nice mapping with fullscreen manageHook. * Whether a window starts out tiled or floating, once toggled fullscreen and back it restores to tiled state. My reading of wm-spec and probably most users expectations is that floats return to float instead. I don't think PositionStore would be needed for this but didn't look into it yet either. audunskaugen, thanks again. If you'd like please send a darcs patch to X.Hooks.EwmhDesktops, or if you want to be credited by some other name you are welcome to email me privately with your preference and I'll include in any patch I prepare. -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings

Comment #6 on issue 339 by audunskaugen: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 The toggling is unsmooth because there are to actions in the process: The WM has to fullscreen the window, and the application itself has to make the content take up the entire window. There is the same kind of glitching with metacity, but it takes a little bit shorter time, so it's less noticable. I agree that the return to tiled state is a bit of a hack, but I cannot think of a way to remember the previous state, short of using PositionStore or an IORef or something ugly like that. It might however be wise to run the returned window through the manage hooks. I'll see about that patch! -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings

Updates: Status: Patch Comment #7 on issue 339 by wirtwo...@gmail.com: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 Indeed on upgrading evince it toggles to and from tiled/fullscreen properly with this event hook. Must have been something messed up with my evince or gnome/dbus that caused it not to work earlier. ooimpress fullscreen continues to be problematic under a variety of xmonad.hs configs. With adobe-flash-10.0.42.34 64-bit splash messages such as "Press esc to leave fullscreen" never time out and fade using this hook, whereas with isFullscreen or alexey fullscreen query and manageHook they do. My xlib knowledge is not enough to know if any other behaviour should be added to the eventHook window handling to help with those issues, but I'll consider them flash and openoffice problems at this point. Any thoughts as to whether this patch should go into contrib as-is vs. wait for some revisions first? My feeling is that it is a good base to work from and adds much- desired functionality for users of gtk_window_fullscreen apps, so is worth adding even without revision. Also that it will be more accessible for xlib hackers to improve being included with contrib rather than being only a patch on tracker/ML/darcswatch. -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings

Comment #8 on issue 339 by nomeata: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 Is anyone pursuing this issue? Proper Fullscreen toggle support would be great!

Comment #9 on issue 339 by nomeata: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 And just for reference, this is the corresponding bug on the Debian bug tracker: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=524281

Comment #10 on issue 339 by audunskaugen: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 This bug is fixed in the darcs version of XMonadContrib, with the fullscreenEventHook from XMonad.Hooks.EwmhDesktops. When it will make it into a release is another question.

Comment #11 on issue 339 by otakuj...@gmail.com: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 I just wanted to mention that comment 4 resolved this issue for me on Ubuntu 10.04. Because it might be useful for others in a similar situation, I've attached my current xmonad.hs, which includes gnome support and the fix for fullscreen flash. Attachments: xmonad.hs 1.6 KB

Updates: Status: Fixed Comment #12 on issue 339 by wirtwo...@gmail.com: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 EwmhDesktops fullscreenEventHook and XMonad.Layout.Fullscreen provide this support in 0.10. Re. what fullscreen support should be provided by default, and what support (if any) should be in core, see Issue 313. Re. changing the fullscreen implementation from doFullFloat to something that ignores some events and is always borderless without using X.Layout.NoBorders.lessBorders, see Issue 410.

Comment #13 on issue 339 by lsgew...@gmail.com: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 The EWMH documentation ought to mention that fullscreenEventHook is _not_ included in the ewmh utility function. (Or it should be included there.)

Comment #14 on issue 339 by nome...@gmail.com: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 Wrong user was logged in, the above comment was by me.

Comment #15 on issue 339 by vogt.a...@gmail.com: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 I've pushed a documentation patch which also shows how you'd use fullscreenEventHook. It may be better to just include fullscreenEventHook in that ewmh function: if there are no objections within a week I'll assume nobody minds that change.

Comment #16 on issue 339 by wirtwo...@gmail.com: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 The only thing swaying me toward not building it into ewmh is hoping to eliminate code duplication. It's a little painful to have so similar a fullscreenEventHook here and in X.Layout.Fullscreen. Been looking into using a similar to ewmh helper to allow replacing the EwmhDesktops one with X.L.Fullscreen version. (Use the X.L.Fullscreen one in handleEventHook and prepend a suitable layout modifier to layoutHook) but there are several problems with that. Prepending to layoutHook isn't always the right thing to do. The NoBorders lessBorders options aside from Never (for multi head users at least) won't remove borders from the layoutModifier versions, nor will manageHook do a borderless fullscreen float for some apps like flash without using the EwmhDesktops version. So at the moment there are a bunch of solutions that work sometimes, but it's looking difficult to find a way to eliminate the code duplication and satisfy most users most of the time. I suspect most people will move from ewmh to individually customizing XConfig hooks, anyway. Or manually toggle to noBorders Full for a few problem apps. So all that said, I hope someone smarter sees a better way, but for now, I reluctantly say okay, add the current EwmhDesktops.fullscreenEventHook to ewmh.

Comment #17 on issue 339 by audunska...@gmail.com: Support for toggling totem movie player fullscreen http://code.google.com/p/xmonad/issues/detail?id=339 I don't have any strong opinion, but one thing to consider is that the fullscreenEventHook in EwmhDesktops and X.L.Fullscreen won't play well together. So if a user blindly uses the ewmh function and then also uses the X.L.Fullscreen module, she will get problems.
participants (1)
-
codesite-noreply@google.com