Hello Philipp, Your inputs made sense, thanks for reviewing. Herewith the updates. As before GPL/MPL is valid.
Let me find another target... Luc 2011/11/19 Philipp Weissenbacher <p.weissenbac...@gmail.com> > Hoi Luc, all, > > Not sure if that really matters, but I think that > // Klammerung, da in diesem Handler das Window zerstoert werden darf > // Parentheses, as in this handler the window could be destroyed > "could" should be "can". > > On line 925 (in the patch): > I would translate it like this: > If the Overlap-Window has changed, we need to test whether > OverlapWindows that had the > Child window as their parent need to be put into the window hierarchy. > > Line 992/993: > I think that > If it is a SystemWindow it automatically pops up on top of all other > windows if needed. > is a bit clearer. > > Line 1083: > I think you meant "was sent once"; I think "... already sent" is more > appropriate. > > Otherwise very nice patch! And let's hang in there and get all the > German comments out! > > Cheers, > Philipp > > On 19 November 2011 00:41, Luc Castermans <luc.casterm...@gmail.com> > wrote: > > Gents, > > > > Triggered by Lionel I attacked this close to 10k lines beast and > translated > > comments from German to English. > > > > My contribution is under GPL/MPL. as my last and future ones. > > > > thanks, > > > > -- > > Luc Castermans > > mailto:luc.casterm...@gmail.com > > > > _______________________________________________ > > LibreOffice mailing list > > LibreOffice@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/libreoffice > > > > > -- Luc Castermans mailto:luc.casterm...@gmail.com
From a16bac0b574e17223971bd5f529d8ad47c5fd54d Mon Sep 17 00:00:00 2001 From: Luc Castermans <luc.casterm...@gmail.com> Date: Sat, 19 Nov 2011 16:47:00 +0100 Subject: [PATCH] corrected input from Philipp on translations. Thanks, --- vcl/source/window/window.cxx | 44 ++++++++++++++++++++--------------------- 1 files changed, 21 insertions(+), 23 deletions(-) diff --git a/vcl/source/window/window.cxx b/vcl/source/window/window.cxx index db2f109..a9c668e 100644 --- a/vcl/source/window/window.cxx +++ b/vcl/source/window/window.cxx @@ -645,7 +645,7 @@ void Window::ImplInitWindowData( WindowType nType ) mpWindowImpl->mnWaitCount = 0; // Wait-Count (>1 == Warte-MousePointer) mpWindowImpl->mnPaintFlags = 0; // Flags for ImplCallPaint mpWindowImpl->mnParentClipMode = 0; // Flags for Parent-ClipChildren-Mode - mpWindowImpl->mnActivateMode = 0; // Wird bei System/Overlap-Windows umgesetzt + mpWindowImpl->mnActivateMode = 0; // Will be converted in System/Overlap-Windows mpWindowImpl->mnDlgCtrlFlags = 0; // DialogControl-Flags mpWindowImpl->mnLockCount = 0; // LockCount mpWindowImpl->meAlwaysInputMode = AlwaysInputNone; // neither AlwaysEnableInput nor AlwaysDisableInput called @@ -771,7 +771,7 @@ void Window::ImplInit( Window* pParent, WinBits nStyle, SystemParentData* pSyste ImplInsertWindow( pParent ); mpWindowImpl->mnStyle = nStyle; - // Overlap-Window-Daten + // Overlap-Window-Data if ( mpWindowImpl->mbOverlapWin ) { mpWindowImpl->mpOverlapData = new ImplOverlapData; @@ -1055,7 +1055,7 @@ void Window::ImplInsertWindow( Window* pParent ) mpWindowImpl->mpNextOverlap = mpWindowImpl->mpFrameData->mpFirstOverlap; mpWindowImpl->mpFrameData->mpFirstOverlap = this; - // Overlap-Windows sind per default die obersten + // Overlap-Windows are per default the upper mpWindowImpl->mpNext = pFirstOverlapParent->mpWindowImpl->mpFirstOverlap; pFirstOverlapParent->mpWindowImpl->mpFirstOverlap = this; if ( !pFirstOverlapParent->mpWindowImpl->mpLastOverlap ) @@ -4647,7 +4647,7 @@ void Window::SimulateKeyPress( sal_uInt16 nKeyCode ) const void Window::MouseMove( const MouseEvent& rMEvt ) { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -4660,7 +4660,7 @@ void Window::MouseMove( const MouseEvent& rMEvt ) void Window::MouseButtonDown( const MouseEvent& rMEvt ) { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -4673,7 +4673,7 @@ void Window::MouseButtonDown( const MouseEvent& rMEvt ) void Window::MouseButtonUp( const MouseEvent& rMEvt ) { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -4686,7 +4686,7 @@ void Window::MouseButtonUp( const MouseEvent& rMEvt ) void Window::KeyInput( const KeyEvent& rKEvt ) { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -4699,7 +4699,7 @@ void Window::KeyInput( const KeyEvent& rKEvt ) void Window::KeyUp( const KeyEvent& rKEvt ) { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -4718,7 +4718,7 @@ void Window::PrePaint() void Window::Paint( const Rectangle& rRect ) { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -4770,7 +4770,7 @@ void Window::Deactivate() void Window::GetFocus() { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -4790,7 +4790,7 @@ void Window::GetFocus() void Window::LoseFocus() { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -4802,7 +4802,7 @@ void Window::LoseFocus() void Window::RequestHelp( const HelpEvent& rHEvt ) { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -4858,7 +4858,7 @@ void Window::RequestHelp( const HelpEvent& rHEvt ) void Window::Command( const CommandEvent& rCEvt ) { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -5006,7 +5006,7 @@ void Window::ImplNotifyKeyMouseCommandEventListeners( NotifyEvent& rNEvt ) long Window::PreNotify( NotifyEvent& rNEvt ) { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -5053,7 +5053,7 @@ long Window::PreNotify( NotifyEvent& rNEvt ) long Window::Notify( NotifyEvent& rNEvt ) { - { // Parentheses, as in this handler the window could be destroyed + { // Parentheses, as in this handler the window can be destroyed DBG_CHKTHIS( Window, ImplDbgCheckWindow ); } @@ -6119,12 +6119,9 @@ void Window::SetParent( Window* pNewParent ) if ( ImplUpdatePos() ) ImplUpdateSysObjPos(); - // if the overlap window was changed, test if - // also overlap windows having the child window as parent should - // be converted in the window hierachy - // - // Original German comment reads: "...in der Window-Hirachie umgesetzt werden muessen" - // not clear how to translate "umgesetzt" in this case + // If the Overlap-Window has changed, we need to test whether + // OverlapWindows that had the Child window as their parent + // need to be put into the window hierarchy. if ( ImplIsOverlapWindow() ) { if ( bNewFrame ) @@ -6328,7 +6325,8 @@ void Window::Show( sal_Bool bVisible, sal_uInt16 nFlags ) // such that these can initialise themselves ImplCallInitShow(); - // if required it will shift frontward wenn it is a SystemWindow, + // If it is a SystemWindow it automatically pops up on top of + // all other windows if needed. if ( ImplIsOverlapWindow() && !(nFlags & SHOW_NOACTIVATE) ) { ImplStartToTop(( nFlags & SHOW_FOREGROUNDTASK ) ? TOTOP_FOREGROUNDTASK : 0 ); @@ -6934,7 +6932,7 @@ void Window::SetZOrder( Window* pRefWindow, sal_uInt16 nFlags ) // for HTML-documents with many controls. If this // check gives problems a flag should be introduced // which checks if the window after Show was - // sent ones + // already sent if ( !bInitWinClipRegion ) { -- 1.7.2.5
_______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice