[Libreoffice-commits] .: 2 commits - filter/source

[Jan Holesovsky]

 filter/source/pdf/impdialog.cxx |    6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

New commits:
commit ef819a24e14274aa21d439c6ae7874879b31c28f
Author: Jan Holesovsky <kendy at suse.cz>
Date:   Fri Oct 7 09:34:18 2011 +0200

    Kill dead code.

diff --git a/filter/source/pdf/impdialog.cxx b/filter/source/pdf/impdialog.cxx
index cb82219..6db5ee9 100644
--- a/filter/source/pdf/impdialog.cxx
+++ b/filter/source/pdf/impdialog.cxx
@@ -649,8 +649,6 @@ SfxTabPage*  ImpPDFTabGeneralPage::Create( Window* pParent,
 IMPL_LINK( ImpPDFTabGeneralPage, TogglePagesHdl, void*, EMPTYARG )
 {
     maEdPages.Enable( maRbRange.IsChecked() );
-    //Sym2_5805, When the control is disabled, it is also readonly. So here, it is not necessary to set it as readonly.
-    //maEdPages.SetReadOnly( !maRbRange.IsChecked() );
     if ( maRbRange.IsChecked() )
         maEdPages.GrabFocus();
     return 0;
@@ -689,8 +687,6 @@ IMPL_LINK( ImpPDFTabGeneralPage, ToggleAddStreamHdl, void*, EMPTYARG )
             maRbRange.Enable( sal_False );
             maRbSelection.Enable( sal_False );
             maEdPages.Enable( sal_False );
-            //Sym2_5805, When the control is disabled, it is also readonly. So here, it is not necessary to set it as readonly.
-            //maEdPages.SetReadOnly( sal_True );
             maRbAll.Enable( sal_False );
         }
         else
commit 6087dc9cddde7052258996389d561f415ebee13c
Author: Maxim Iorsh <iorsh at users.sourceforge.net>
Date:   Wed Oct 5 22:57:27 2011 +0200

    filter: PDF Export UI "page range" autofocus
    
    When the user selects "Pages" radio button in the Range section, it is very
    reasonable to expect that she would now want to specify the range. Thus moving
    the focus automatically to the page range edit box would save the user a mouse
    click.a
    
    Code is contributed under the LGPLv3+ / MPL.

diff --git a/filter/source/pdf/impdialog.cxx b/filter/source/pdf/impdialog.cxx
index 67ba921..cb82219 100644
--- a/filter/source/pdf/impdialog.cxx
+++ b/filter/source/pdf/impdialog.cxx
@@ -651,6 +651,8 @@ IMPL_LINK( ImpPDFTabGeneralPage, TogglePagesHdl, void*, EMPTYARG )
     maEdPages.Enable( maRbRange.IsChecked() );
     //Sym2_5805, When the control is disabled, it is also readonly. So here, it is not necessary to set it as readonly.
     //maEdPages.SetReadOnly( !maRbRange.IsChecked() );
+    if ( maRbRange.IsChecked() )
+        maEdPages.GrabFocus();
     return 0;
 }