<div>Great! Thanks for reviewing and applying this patch.</div>
<div>You may also want to have a look at a similar patch for wingdi module, which I have posted to vlc-devel mail-list on May 3.<br><br> </div>
<div><span class="gmail_quote">On 5/8/07, <b class="gmail_sendername">Damien Fouilleul</b> <<a href="mailto:damien.fouilleul@laposte.net">damien.fouilleul@laposte.net</a>> wrote:</span>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">Sergey Volk wrote:<br>><br>> Hi,<br>> Current implementation of directx video output modules (directx,
<br>> direct3d, glwin32) leaves main interface window visible when the user<br>> toggles fullscreen mode. This causes two problems: (a) the user sees<br>> empty main interface window when he switches to another application or
<br>> cycles through existing windows using Alt-Tab and (b) main interface<br>> window overlaps video window when he toggles "video-on-top" option<br>> while fullscreen mode is active. In order to fix both these issues, we
<br>> should hide main interface window when entering fullscreen mode and<br>> restore it when leaving fullscreen mode. But some GUI modules (in<br>> particular qt4 module) may need to call show() for main interface
<br>> window in order to apply "video-on-top" flag, so in order to avoid<br>> condition (b) described earlier we need also to hide<br>> VOUT_SET_STAY_ON_TOP queries from main interface window while we are
<br>> in fullscreen mode and update "video-on-top" status for main window<br>> when we are leaving fullscreen mode.<br>><br>> I have also noticed that there is a code in<br>> src/video_output/vout_intf.c:FullscreenCallback (line 1151), which
<br>> sets video-on-top variable to false, when entering fullscreen mode. I<br>> think this code should be removed, there is no point in disabling<br>> video-on-top option, as the user may change this option in fullscreen
<br>> mode anyway (through mouse / hotkey controls, at least). So<br>> video_output modules should be aware that video-on-top variable may be<br>> toggled in fullscreen mode and should handle such situations<br>> appropriately. I'm developing under win32/cygwin, so I haven't checked
<br>> if similar reasoning applies to X11 module or any of the other video<br>> output modules. It seems that at least wingdi output module has the<br>> same problem, but it's code is somewhat different from<br>
> DirectX/Direct3D/Win32 OpenGL code, so I'll provide patch for wingdi<br>> module later. In the meantime, I think it is safe to leave that old<br>> code as it is.<br>><br>> Two patches are attached to this message: 
vlc-vout-dx-fullscreen.patch<br>> is a version which fixes fullscreen mode for DirectX module only, it<br>> is intended only to demonstrate what changes are being applied to<br>> current version. Another one - vlc-vout-dx-fullscreen2.patch
, does<br>> essentially the same, only it fixes all three modules (directx,<br>> direct3d and opengl) and refactors some common code for toggling<br>> fullscreen mode into Win32ToggleFullscreen function, to remove code
<br>> duplication between those modules.<br>><br>patch applied thanks !<br><br>--<br>This is the vlc-devel mailing-list, see <a href="http://www.videolan.org/vlc/">http://www.videolan.org/vlc/</a><br>To unsubscribe, please read 
<a href="http://developers.videolan.org/lists.html">http://developers.videolan.org/lists.html</a><br><br></blockquote></div><br>