[Libreoffice-commits] core.git: Branch 'libreoffice-6-1' - sw/source sw/uiconfig

Caolán McNamara caolanm at redhat.com
Sat Jun 16 13:47:22 UTC 2018


 sw/source/ui/chrdlg/numpara.cxx       |  267 +++++++++++++++-------------------
 sw/source/ui/chrdlg/pardlg.cxx        |    4 
 sw/source/ui/fmtui/tmpdlg.cxx         |    4 
 sw/source/uibase/inc/numpara.hxx      |   51 +++---
 sw/uiconfig/swriter/ui/numparapage.ui |   16 +-
 5 files changed, 156 insertions(+), 186 deletions(-)

New commits:
commit 2d0ac1a3581088f7f779c7ad6b0f9f80026ee44b
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Fri Jun 15 10:51:30 2018 +0100

    weld SwParagraphNumTabPage
    
    Change-Id: I6d95b5235d43dabc17515f3ac31063093c68d959
    Reviewed-on: https://gerrit.libreoffice.org/55875
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/source/ui/chrdlg/numpara.cxx b/sw/source/ui/chrdlg/numpara.cxx
index 589e808fe4e7..fda0958c26a7 100644
--- a/sw/source/ui/chrdlg/numpara.cxx
+++ b/sw/source/ui/chrdlg/numpara.cxx
@@ -42,40 +42,36 @@ const sal_uInt16 SwParagraphNumTabPage::aPageRg[] = {
     0
 };
 
-SwParagraphNumTabPage::SwParagraphNumTabPage(vcl::Window* pParent, const SfxItemSet& rAttr ) :
-    SfxTabPage(pParent, "NumParaPage", "modules/swriter/ui/numparapage.ui", &rAttr),
-
-    msOutlineNumbering( SwResId( STR_OUTLINE_NUMBERING ) ),
-    bModified(false),
-    bCurNumrule(false)
+SwParagraphNumTabPage::SwParagraphNumTabPage(TabPageParent pParent, const SfxItemSet& rAttr)
+    : SfxTabPage(pParent, "modules/swriter/ui/numparapage.ui", "NumParaPage", &rAttr)
+    , msOutlineNumbering(SwResId(STR_OUTLINE_NUMBERING ))
+    , bModified(false)
+    , bCurNumrule(false)
+    , m_xOutlineStartBX(m_xBuilder->weld_widget("boxOUTLINE"))
+    , m_xOutlineLvLB(m_xBuilder->weld_combo_box_text("comboLB_OUTLINE_LEVEL"))
+    , m_xNumberStyleBX(m_xBuilder->weld_widget("boxNUMBER_STYLE"))
+    , m_xNumberStyleLB(m_xBuilder->weld_combo_box_text("comboLB_NUMBER_STYLE"))
+    , m_xEditNumStyleBtn(m_xBuilder->weld_button("editnumstyle"))
+    , m_xNewStartCB(m_xBuilder->weld_check_button("checkCB_NEW_START"))
+    , m_xNewStartBX(m_xBuilder->weld_widget("boxNEW_START"))
+    , m_xNewStartNumberCB(m_xBuilder->weld_check_button("checkCB_NUMBER_NEW_START"))
+    , m_xNewStartNF(m_xBuilder->weld_spin_button("spinNF_NEW_START"))
+    , m_xCountParaFram(m_xBuilder->weld_widget("frameFL_COUNT_PARA"))
+    , m_xCountParaCB(m_xBuilder->weld_check_button("checkCB_COUNT_PARA"))
+    , m_xRestartParaCountCB(m_xBuilder->weld_check_button("checkCB_RESTART_PARACOUNT"))
+    , m_xRestartBX(m_xBuilder->weld_widget("boxRESTART_NO"))
+    , m_xRestartNF(m_xBuilder->weld_spin_button("spinNF_RESTART_PARA"))
 {
-    get(m_pOutlineStartBX,         "boxOUTLINE");
-    get(m_pOutlineLvLB,            "comboLB_OUTLINE_LEVEL");
-
-    get(m_pNumberStyleBX,          "boxNUMBER_STYLE");
-    get(m_pNumberStyleLB,          "comboLB_NUMBER_STYLE");
-    get(m_pEditNumStyleBtn,        "editnumstyle");
-    get(m_pNewStartBX,             "boxNEW_START");
-    get(m_pNewStartCB,             "checkCB_NEW_START");
-    m_pNewStartCB->SetState(TRISTATE_FALSE);
-    get(m_pNewStartNumberCB,       "checkCB_NUMBER_NEW_START");
-    m_pNewStartNumberCB->SetState(TRISTATE_FALSE);
-    get(m_pNewStartNF,             "spinNF_NEW_START");
-
-    get(m_pCountParaFram,          "frameFL_COUNT_PARA");
-    get(m_pCountParaCB,            "checkCB_COUNT_PARA");
-    m_pCountParaCB->SetState(TRISTATE_FALSE);
-    get(m_pRestartParaCountCB,     "checkCB_RESTART_PARACOUNT");
-    m_pRestartParaCountCB->SetState(TRISTATE_FALSE);
-
-    get(m_pRestartBX,              "boxRESTART_NO");
-    get(m_pRestartNF,              "spinNF_RESTART_PARA");
-
-    sal_Int32 numSelectPos = m_pNumberStyleLB->GetSelectedEntryPos();
-    if ( numSelectPos == 0 )
-        m_pEditNumStyleBtn->Disable();
+    m_xNewStartCB->set_state(TRISTATE_FALSE);
+    m_xNewStartNumberCB->set_state(TRISTATE_FALSE);
+    m_xCountParaCB->set_state(TRISTATE_FALSE);
+    m_xRestartParaCountCB->set_state(TRISTATE_FALSE);
+
+    int numSelectPos = m_xNumberStyleLB->get_active();
+    if (numSelectPos == 0)
+        m_xEditNumStyleBtn->set_sensitive(false);
     else
-        m_pEditNumStyleBtn->Enable();
+        m_xEditNumStyleBtn->set_sensitive(true);
 
     const SfxPoolItem* pItem;
     SfxObjectShell* pObjSh;
@@ -86,53 +82,32 @@ SwParagraphNumTabPage::SwParagraphNumTabPage(vcl::Window* pParent, const SfxItem
         const sal_uInt16 nHtmlMode = static_cast<const SfxUInt16Item*>(pItem)->GetValue();
 
         if (HTMLMODE_ON & nHtmlMode)
-            m_pCountParaFram->Hide();
+            m_xCountParaFram->hide();
     }
 
-    m_pNewStartCB->SetClickHdl(LINK(this, SwParagraphNumTabPage, NewStartHdl_Impl));
-    m_pNewStartNumberCB->SetClickHdl(LINK(this, SwParagraphNumTabPage, NewStartHdl_Impl));
-    m_pNumberStyleLB->SetSelectHdl(LINK(this, SwParagraphNumTabPage, StyleHdl_Impl));
-    m_pCountParaCB->SetClickHdl(LINK(this, SwParagraphNumTabPage, LineCountHdl_Impl));
-    m_pRestartParaCountCB->SetClickHdl( LINK(this, SwParagraphNumTabPage, LineCountHdl_Impl));
-    m_pNumberStyleLB->SetSelectHdl( LINK( this, SwParagraphNumTabPage, EditNumStyleSelectHdl_Impl ) );
-    m_pEditNumStyleBtn->SetClickHdl( LINK(this, SwParagraphNumTabPage, EditNumStyleHdl_Impl));
+    m_xNewStartCB->connect_toggled(LINK(this, SwParagraphNumTabPage, NewStartHdl_Impl));
+    m_xNewStartNumberCB->connect_toggled(LINK(this, SwParagraphNumTabPage, NewStartHdl_Impl));
+    m_xNumberStyleLB->connect_changed(LINK(this, SwParagraphNumTabPage, StyleHdl_Impl));
+    m_xCountParaCB->connect_toggled(LINK(this, SwParagraphNumTabPage, LineCountHdl_Impl));
+    m_xRestartParaCountCB->connect_toggled(LINK(this, SwParagraphNumTabPage, LineCountHdl_Impl));
+    m_xNumberStyleLB->connect_changed(LINK(this, SwParagraphNumTabPage, EditNumStyleSelectHdl_Impl));
+    m_xEditNumStyleBtn->connect_clicked(LINK(this, SwParagraphNumTabPage, EditNumStyleHdl_Impl));
 }
 
 SwParagraphNumTabPage::~SwParagraphNumTabPage()
 {
-    disposeOnce();
-}
-
-void SwParagraphNumTabPage::dispose()
-{
-    m_pOutlineStartBX.clear();
-    m_pOutlineLvLB.clear();
-    m_pNumberStyleBX.clear();
-    m_pNumberStyleLB.clear();
-    m_pEditNumStyleBtn.clear();
-    m_pNewStartCB.clear();
-    m_pNewStartBX.clear();
-    m_pNewStartNumberCB.clear();
-    m_pNewStartNF.clear();
-    m_pCountParaFram.clear();
-    m_pCountParaCB.clear();
-    m_pRestartParaCountCB.clear();
-    m_pRestartBX.clear();
-    m_pRestartNF.clear();
-    SfxTabPage::dispose();
 }
 
-VclPtr<SfxTabPage> SwParagraphNumTabPage::Create( TabPageParent pParent,
-                                                  const SfxItemSet* rSet )
+VclPtr<SfxTabPage> SwParagraphNumTabPage::Create(TabPageParent pParent, const SfxItemSet* rSet)
 {
-    return VclPtr<SwParagraphNumTabPage>::Create(pParent.pParent, *rSet);
+    return VclPtr<SwParagraphNumTabPage>::Create(pParent, *rSet);
 }
 
 bool SwParagraphNumTabPage::FillItemSet( SfxItemSet* rSet )
 {
-    if( m_pOutlineLvLB->IsValueChangedFromSaved())
+    if (m_xOutlineLvLB->get_value_changed_from_saved())
     {
-        const sal_uInt16 aOutlineLv = m_pOutlineLvLB->GetSelectedEntryPos();
+        const sal_uInt16 aOutlineLv = m_xOutlineLvLB->get_active();
         const SfxUInt16Item* pOldOutlineLv = static_cast<const SfxUInt16Item*>(GetOldItem( *rSet, SID_ATTR_PARA_OUTLINE_LEVEL));
         if (pOldOutlineLv)
         {
@@ -144,11 +119,11 @@ bool SwParagraphNumTabPage::FillItemSet( SfxItemSet* rSet )
         }
     }
 
-    if( m_pNumberStyleLB->IsValueChangedFromSaved())
+    if (m_xNumberStyleLB->get_value_changed_from_saved())
     {
         OUString aStyle;
-        if(m_pNumberStyleLB->GetSelectedEntryPos())
-            aStyle = m_pNumberStyleLB->GetSelectedEntry();
+        if (m_xNumberStyleLB->get_active())
+            aStyle = m_xNumberStyleLB->get_active_text();
         const SfxStringItem* pOldRule = static_cast<const SfxStringItem*>(GetOldItem( *rSet, SID_ATTR_PARA_NUMRULE));
         SfxStringItem* pRule = pOldRule ? static_cast<SfxStringItem*>(pOldRule->Clone()) : nullptr;
         if (pRule)
@@ -159,26 +134,26 @@ bool SwParagraphNumTabPage::FillItemSet( SfxItemSet* rSet )
             bModified = true;
         }
     }
-    if(m_pNewStartCB->IsValueChangedFromSaved() ||
-       m_pNewStartNumberCB->IsValueChangedFromSaved()||
-       m_pNewStartNF->IsValueChangedFromSaved())
+    if (m_xNewStartCB->get_state_changed_from_saved() ||
+        m_xNewStartNumberCB->get_state_changed_from_saved()||
+        m_xNewStartNF->get_value_changed_from_saved())
     {
         bModified = true;
-        bool bNewStartChecked = TRISTATE_TRUE == m_pNewStartCB->GetState();
-        bool bNumberNewStartChecked = TRISTATE_TRUE == m_pNewStartNumberCB->GetState();
+        bool bNewStartChecked = TRISTATE_TRUE == m_xNewStartCB->get_state();
+        bool bNumberNewStartChecked = TRISTATE_TRUE == m_xNewStartNumberCB->get_state();
         rSet->Put(SfxBoolItem(FN_NUMBER_NEWSTART, bNewStartChecked));
         rSet->Put(SfxUInt16Item(FN_NUMBER_NEWSTART_AT,
-                  bNumberNewStartChecked && bNewStartChecked ? static_cast<sal_uInt16>(m_pNewStartNF->GetValue()) : USHRT_MAX));
+                  bNumberNewStartChecked && bNewStartChecked ? static_cast<sal_uInt16>(m_xNewStartNF->get_value()) : USHRT_MAX));
     }
 
-    if(m_pCountParaCB->IsValueChangedFromSaved()||
-       m_pRestartParaCountCB->IsValueChangedFromSaved() ||
-       m_pRestartNF->IsValueChangedFromSaved() )
+    if (m_xCountParaCB->get_state_changed_from_saved()||
+        m_xRestartParaCountCB->get_state_changed_from_saved() ||
+        m_xRestartNF->get_value_changed_from_saved())
     {
         SwFormatLineNumber aFormat;
-        aFormat.SetStartValue( static_cast< sal_uLong >(m_pRestartParaCountCB->GetState() == TRISTATE_TRUE ?
-                                m_pRestartNF->GetValue() : 0 ));
-        aFormat.SetCountLines( m_pCountParaCB->IsChecked() );
+        aFormat.SetStartValue( static_cast< sal_uLong >(m_xRestartParaCountCB->get_state() == TRISTATE_TRUE ?
+                                m_xRestartNF->get_value() : 0 ));
+        aFormat.SetCountLines(m_xCountParaCB->get_active());
         rSet->Put(aFormat);
         bModified = true;
     }
@@ -187,15 +162,16 @@ bool SwParagraphNumTabPage::FillItemSet( SfxItemSet* rSet )
 
 void SwParagraphNumTabPage::ChangesApplied()
 {
-    m_pOutlineLvLB->SaveValue();
-    m_pNumberStyleLB->SaveValue();
-    m_pNewStartCB->SaveValue();
-    m_pNewStartNumberCB->SaveValue();
-    m_pCountParaCB->SaveValue();
-    m_pRestartParaCountCB->SaveValue();
-    m_pRestartNF->SaveValue();
+    m_xOutlineLvLB->save_value();
+    m_xNumberStyleLB->save_value();
+    m_xNewStartCB->save_state();
+    m_xNewStartNumberCB->save_state();
+    m_xCountParaCB->save_state();
+    m_xRestartParaCountCB->save_state();
+    m_xRestartNF->save_value();
 }
-void    SwParagraphNumTabPage::Reset(const SfxItemSet* rSet)
+
+void SwParagraphNumTabPage::Reset(const SfxItemSet* rSet)
 {
     bool bHasNumberStyle = false;
 
@@ -205,13 +181,13 @@ void    SwParagraphNumTabPage::Reset(const SfxItemSet* rSet)
     if( eItemState >= SfxItemState::DEFAULT )
     {
         nOutlineLv = static_cast<const SfxUInt16Item &>(rSet->Get( GetWhich(SID_ATTR_PARA_OUTLINE_LEVEL) )).GetValue();
-        m_pOutlineLvLB->SelectEntryPos( nOutlineLv ) ;
+        m_xOutlineLvLB->set_active(nOutlineLv) ;
     }
     else
     {
-        m_pOutlineLvLB->SetNoSelection();
+        m_xOutlineLvLB->set_active(-1);
     }
-    m_pOutlineLvLB->SaveValue();
+    m_xOutlineLvLB->save_value();
 
     eItemState = rSet->GetItemState( GetWhich(SID_ATTR_PARA_NUMRULE) );
 
@@ -219,26 +195,27 @@ void    SwParagraphNumTabPage::Reset(const SfxItemSet* rSet)
     {
         OUString aStyle = static_cast<const SfxStringItem &>(rSet->Get( GetWhich(SID_ATTR_PARA_NUMRULE) )).GetValue();
         if(aStyle.isEmpty())
-            aStyle = m_pNumberStyleLB->GetEntry(0);
+            aStyle = m_xNumberStyleLB->get_text(0);
 
         if( aStyle == "Outline")
         {
-            m_pNumberStyleLB->InsertEntry( msOutlineNumbering );
-            m_pNumberStyleLB->SelectEntry( msOutlineNumbering );
-            m_pNumberStyleLB->RemoveEntry(msOutlineNumbering);
-            m_pNumberStyleLB->SaveValue();
+            //add it, select it, remove it ? do we really want set_active(-1) instead ?
+            m_xNumberStyleLB->append_text(msOutlineNumbering);
+            m_xNumberStyleLB->set_active_text(msOutlineNumbering);
+            m_xNumberStyleLB->remove_text(msOutlineNumbering);
+            m_xNumberStyleLB->save_value();
         }
         else
-            m_pNumberStyleLB->SelectEntry( aStyle );
+            m_xNumberStyleLB->set_active_text(aStyle);
 
         bHasNumberStyle = true;
     }
     else
     {
-        m_pNumberStyleLB->SetNoSelection();
+        m_xNumberStyleLB->set_active(-1);
     }
 
-    m_pNumberStyleLB->SaveValue();
+    m_xNumberStyleLB->save_value();
 
     eItemState = rSet->GetItemState( FN_NUMBER_NEWSTART );
     if(eItemState > SfxItemState::DEFAULT )
@@ -246,99 +223,93 @@ void    SwParagraphNumTabPage::Reset(const SfxItemSet* rSet)
         bCurNumrule = true;
         const SfxBoolItem& rStart = static_cast<const SfxBoolItem&>(rSet->Get(FN_NUMBER_NEWSTART));
 
-        m_pNewStartCB->SetState(rStart.GetValue() ? TRISTATE_TRUE : TRISTATE_FALSE );
-
-        m_pNewStartCB->EnableTriState(false);
+        m_xNewStartCB->set_state(rStart.GetValue() ? TRISTATE_TRUE : TRISTATE_FALSE );
     }
     else
-        m_pNewStartCB->SetState(bHasNumberStyle ? TRISTATE_FALSE : TRISTATE_INDET);
+        m_xNewStartCB->set_state(bHasNumberStyle ? TRISTATE_FALSE : TRISTATE_INDET);
 
-    m_pNewStartCB->SaveValue();
+    m_xNewStartCB->save_state();
 
     eItemState = rSet->GetItemState( FN_NUMBER_NEWSTART_AT);
     if( eItemState > SfxItemState::DEFAULT )
     {
         const sal_uInt16 nNewStart = static_cast<const SfxUInt16Item&>(rSet->Get(FN_NUMBER_NEWSTART_AT)).GetValue();
         const bool bNotMax = USHRT_MAX != nNewStart;
-        m_pNewStartNumberCB->Check(bNotMax);
-        m_pNewStartNF->SetValue(bNotMax ? nNewStart : 1);
-        m_pNewStartNumberCB->EnableTriState(false);
+        m_xNewStartNumberCB->set_active(bNotMax);
+        m_xNewStartNF->set_value(bNotMax ? nNewStart : 1);
     }
     else
-        m_pNewStartCB->SetState(TRISTATE_INDET);
-    NewStartHdl_Impl(m_pNewStartCB);
-    m_pNewStartNF->SaveValue();
-    m_pNewStartNumberCB->SaveValue();
-    StyleHdl_Impl(*m_pNumberStyleLB.get());
+        m_xNewStartCB->set_state(TRISTATE_INDET);
+    NewStartHdl_Impl(*m_xNewStartCB);
+    m_xNewStartNF->save_value();
+    m_xNewStartNumberCB->save_state();
+    StyleHdl_Impl(*m_xNumberStyleLB.get());
     if( SfxItemState::DEFAULT <= rSet->GetItemState(RES_LINENUMBER))
     {
         const SwFormatLineNumber& rNum = rSet->Get(RES_LINENUMBER);
         sal_uLong nStartValue = rNum.GetStartValue();
         bool bCount = rNum.IsCount();
-        m_pCountParaCB->SetState( bCount ? TRISTATE_TRUE : TRISTATE_FALSE );
-        m_pRestartParaCountCB->SetState( 0 != nStartValue ? TRISTATE_TRUE : TRISTATE_FALSE );
-        m_pRestartNF->SetValue(nStartValue == 0 ? 1 : nStartValue);
-        LineCountHdl_Impl(m_pCountParaCB);
-        m_pCountParaCB->EnableTriState(false);
-        m_pRestartParaCountCB->EnableTriState(false);
+        m_xCountParaCB->set_state(bCount ? TRISTATE_TRUE : TRISTATE_FALSE);
+        m_xRestartParaCountCB->set_state(0 != nStartValue ? TRISTATE_TRUE : TRISTATE_FALSE);
+        m_xRestartNF->set_value(nStartValue == 0 ? 1 : nStartValue);
+        LineCountHdl_Impl(*m_xCountParaCB);
     }
     else
     {
-        m_pCountParaCB->SetState(TRISTATE_INDET);
-        m_pRestartParaCountCB->SetState(TRISTATE_INDET);
+        m_xCountParaCB->set_state(TRISTATE_INDET);
+        m_xRestartParaCountCB->set_state(TRISTATE_INDET);
     }
-    m_pCountParaCB->SaveValue();
-    m_pRestartParaCountCB->SaveValue();
-    m_pRestartNF->SaveValue();
+    m_xCountParaCB->save_state();
+    m_xRestartParaCountCB->save_state();
+    m_xRestartNF->save_value();
 
     bModified = false;
 }
 
 void SwParagraphNumTabPage::DisableOutline()
 {
-    m_pOutlineStartBX->Disable();
+    m_xOutlineStartBX->set_sensitive(false);
 }
 
 void SwParagraphNumTabPage::DisableNumbering()
 {
-    m_pNumberStyleBX->Disable();
+    m_xNumberStyleBX->set_sensitive(false);
 }
 
 void SwParagraphNumTabPage::EnableNewStart()
 {
-    m_pNewStartCB->Show();
-    m_pNewStartBX->Show();
+    m_xNewStartCB->show();
+    m_xNewStartBX->show();
 }
 
-IMPL_LINK_NOARG(SwParagraphNumTabPage, NewStartHdl_Impl, Button*, void)
+IMPL_LINK_NOARG(SwParagraphNumTabPage, NewStartHdl_Impl, weld::ToggleButton&, void)
 {
-    bool bEnable = m_pNewStartCB->IsChecked();
-    m_pNewStartNumberCB->Enable(bEnable);
-    m_pNewStartNF->Enable(bEnable && m_pNewStartNumberCB->IsChecked());
+    bool bEnable = m_xNewStartCB->get_active();
+    m_xNewStartNumberCB->set_sensitive(bEnable);
+    m_xNewStartNF->set_sensitive(bEnable && m_xNewStartNumberCB->get_active());
 }
 
-
-IMPL_LINK_NOARG(SwParagraphNumTabPage, LineCountHdl_Impl, Button*, void)
+IMPL_LINK_NOARG(SwParagraphNumTabPage, LineCountHdl_Impl, weld::ToggleButton&, void)
 {
-    m_pRestartParaCountCB->Enable(m_pCountParaCB->IsChecked());
+    m_xRestartParaCountCB->set_sensitive(m_xCountParaCB->get_active());
 
-    bool bEnableRestartValue = m_pRestartParaCountCB->IsEnabled() &&
-                               m_pRestartParaCountCB->IsChecked();
-    m_pRestartBX->Enable(bEnableRestartValue);
+    bool bEnableRestartValue = m_xRestartParaCountCB->get_sensitive() &&
+                               m_xRestartParaCountCB->get_active();
+    m_xRestartBX->set_sensitive(bEnableRestartValue);
 }
 
-IMPL_LINK_NOARG( SwParagraphNumTabPage, EditNumStyleSelectHdl_Impl, ListBox&, void )
+IMPL_LINK_NOARG(SwParagraphNumTabPage, EditNumStyleSelectHdl_Impl, weld::ComboBoxText&, void)
 {
-    sal_Int32 numSelectPos = m_pNumberStyleLB->GetSelectedEntryPos();
-    if ( numSelectPos == 0 )
-        m_pEditNumStyleBtn->Disable();
+    int numSelectPos = m_xNumberStyleLB->get_active();
+    if (numSelectPos == 0)
+        m_xEditNumStyleBtn->set_sensitive(false);
     else
-        m_pEditNumStyleBtn->Enable();
+        m_xEditNumStyleBtn->set_sensitive(true);
 }
 
-IMPL_LINK_NOARG(SwParagraphNumTabPage, EditNumStyleHdl_Impl, Button*, void)
+IMPL_LINK_NOARG(SwParagraphNumTabPage, EditNumStyleHdl_Impl, weld::Button&, void)
 {
-    OUString aTemplName(m_pNumberStyleLB->GetSelectedEntry());
+    OUString aTemplName(m_xNumberStyleLB->get_active_text());
     ExecuteEditNumStyle_Impl( SID_STYLE_EDIT, aTemplName, SfxStyleFamily::Pseudo );
 }
 
@@ -366,11 +337,11 @@ bool SwParagraphNumTabPage::ExecuteEditNumStyle_Impl(
 
 }
 
-IMPL_LINK( SwParagraphNumTabPage, StyleHdl_Impl, ListBox&, rBox, void )
+IMPL_LINK(SwParagraphNumTabPage, StyleHdl_Impl, weld::ComboBoxText&, rBox, void)
 {
-    bool bEnable = bCurNumrule || rBox.GetSelectedEntryPos() > 0;
-    m_pNewStartCB->Enable(bEnable);
-    NewStartHdl_Impl(m_pNewStartCB);
+    bool bEnable = bCurNumrule || rBox.get_active() > 0;
+    m_xNewStartCB->set_sensitive(bEnable);
+    NewStartHdl_Impl(*m_xNewStartCB);
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sw/source/ui/chrdlg/pardlg.cxx b/sw/source/ui/chrdlg/pardlg.cxx
index 01491a1f6d96..8bf2dba3ac1e 100644
--- a/sw/source/ui/chrdlg/pardlg.cxx
+++ b/sw/source/ui/chrdlg/pardlg.cxx
@@ -221,7 +221,7 @@ void SwParaDlg::PageCreated(sal_uInt16 nId, SfxTabPage& rPage)
         }
 
         static_cast<SwParagraphNumTabPage&>(rPage).EnableNewStart();
-        ListBox & rBox = static_cast<SwParagraphNumTabPage&>(rPage).GetStyleBox();
+        weld::ComboBoxText& rBox = static_cast<SwParagraphNumTabPage&>(rPage).GetStyleBox();
         SfxStyleSheetBasePool* pPool = rView.GetDocShell()->GetStyleSheetPool();
         pPool->SetSearchMask(SfxStyleFamily::Pseudo);
         const SfxStyleSheetBase* pBase = pPool->First();
@@ -232,7 +232,7 @@ void SwParaDlg::PageCreated(sal_uInt16 nId, SfxTabPage& rPage)
             pBase = pPool->Next();
         }
         for(std::set<OUString>::const_iterator it = aNames.begin(); it != aNames.end(); ++it)
-            rBox.InsertEntry(*it);
+            rBox.append_text(*it);
     }
     // inits for Area and Transparency TabPages
     // The selection attribute lists (XPropertyList derivates, e.g. XColorList for
diff --git a/sw/source/ui/fmtui/tmpdlg.cxx b/sw/source/ui/fmtui/tmpdlg.cxx
index 500fa8194919..3a0d664855c3 100644
--- a/sw/source/ui/fmtui/tmpdlg.cxx
+++ b/sw/source/ui/fmtui/tmpdlg.cxx
@@ -412,7 +412,7 @@ void SwTemplateDlg::PageCreated( sal_uInt16 nId, SfxTabPage &rPage )
             static_cast<SwParagraphNumTabPage&>(rPage).DisableOutline() ;
             static_cast<SwParagraphNumTabPage&>(rPage).DisableNumbering();
         }//<-end
-        ListBox & rBox = static_cast<SwParagraphNumTabPage&>(rPage).GetStyleBox();
+        weld::ComboBoxText& rBox = static_cast<SwParagraphNumTabPage&>(rPage).GetStyleBox();
         SfxStyleSheetBasePool* pPool = pWrtShell->GetView().GetDocShell()->GetStyleSheetPool();
         pPool->SetSearchMask(SfxStyleFamily::Pseudo);
         const SfxStyleSheetBase* pBase = pPool->First();
@@ -423,7 +423,7 @@ void SwTemplateDlg::PageCreated( sal_uInt16 nId, SfxTabPage &rPage )
             pBase = pPool->Next();
         }
         for(std::set<OUString>::const_iterator it = aNames.begin(); it != aNames.end(); ++it)
-            rBox.InsertEntry(*it);
+            rBox.append_text(*it);
     }
     else if (nId == m_nAlignId)
     {
diff --git a/sw/source/uibase/inc/numpara.hxx b/sw/source/uibase/inc/numpara.hxx
index f0f78a271361..f1ba2dc7a5a2 100644
--- a/sw/source/uibase/inc/numpara.hxx
+++ b/sw/source/uibase/inc/numpara.hxx
@@ -29,35 +29,35 @@
 // are performed.
 class SwParagraphNumTabPage final : public SfxTabPage
 {
-    VclPtr<VclHBox>                 m_pOutlineStartBX;
-    VclPtr<ListBox>                 m_pOutlineLvLB;
-    VclPtr<VclHBox>                 m_pNumberStyleBX;
-    VclPtr<ListBox>                 m_pNumberStyleLB;
-    VclPtr<PushButton>              m_pEditNumStyleBtn;
-
-    VclPtr<TriStateBox>             m_pNewStartCB;
-    VclPtr<VclHBox>                 m_pNewStartBX;
-    VclPtr<TriStateBox>             m_pNewStartNumberCB;
-    VclPtr<NumericField>            m_pNewStartNF;
-
-    VclPtr<VclFrame>                m_pCountParaFram;
-    VclPtr<TriStateBox>             m_pCountParaCB;
-    VclPtr<TriStateBox>             m_pRestartParaCountCB;
-
-    VclPtr<VclHBox>                 m_pRestartBX;
-    VclPtr<NumericField>            m_pRestartNF;
-
     // #outlinelevel#
     const OUString msOutlineNumbering;
 
     bool                    bModified : 1;
     bool                    bCurNumrule : 1;
 
-    DECL_LINK(NewStartHdl_Impl, Button*, void);
-    DECL_LINK( StyleHdl_Impl, ListBox&,void );
-    DECL_LINK(LineCountHdl_Impl, Button*, void);
-    DECL_LINK(EditNumStyleHdl_Impl, Button*, void);
-    DECL_LINK(EditNumStyleSelectHdl_Impl, ListBox&, void);
+    std::unique_ptr<weld::Widget> m_xOutlineStartBX;
+    std::unique_ptr<weld::ComboBoxText> m_xOutlineLvLB;
+    std::unique_ptr<weld::Widget> m_xNumberStyleBX;
+    std::unique_ptr<weld::ComboBoxText> m_xNumberStyleLB;
+    std::unique_ptr<weld::Button> m_xEditNumStyleBtn;
+
+    std::unique_ptr<weld::CheckButton> m_xNewStartCB;
+    std::unique_ptr<weld::Widget> m_xNewStartBX;
+    std::unique_ptr<weld::CheckButton> m_xNewStartNumberCB;
+    std::unique_ptr<weld::SpinButton> m_xNewStartNF;
+
+    std::unique_ptr<weld::Widget> m_xCountParaFram;
+    std::unique_ptr<weld::CheckButton> m_xCountParaCB;
+    std::unique_ptr<weld::CheckButton> m_xRestartParaCountCB;
+
+    std::unique_ptr<weld::Widget> m_xRestartBX;
+    std::unique_ptr<weld::SpinButton> m_xRestartNF;
+
+    DECL_LINK(NewStartHdl_Impl, weld::ToggleButton&, void);
+    DECL_LINK(StyleHdl_Impl, weld::ComboBoxText&,void);
+    DECL_LINK(LineCountHdl_Impl, weld::ToggleButton&, void);
+    DECL_LINK(EditNumStyleHdl_Impl, weld::Button&, void);
+    DECL_LINK(EditNumStyleSelectHdl_Impl, weld::ComboBoxText&, void);
 
     static const sal_uInt16 aPageRg[];
 
@@ -65,9 +65,8 @@ class SwParagraphNumTabPage final : public SfxTabPage
                           SfxStyleFamily nFamily );
 
 public:
-    SwParagraphNumTabPage(vcl::Window* pParent, const SfxItemSet& rSet );
+    SwParagraphNumTabPage(TabPageParent pParent, const SfxItemSet& rSet );
     virtual ~SwParagraphNumTabPage() override;
-    virtual void        dispose() override;
 
     static VclPtr<SfxTabPage> Create( TabPageParent pParent,
                                       const SfxItemSet* rSet );
@@ -81,7 +80,7 @@ public:
     void                DisableOutline();
     void                DisableNumbering();
 
-    ListBox&            GetStyleBox() {return *m_pNumberStyleLB;};
+    weld::ComboBoxText& GetStyleBox() {return *m_xNumberStyleLB;};
 };
 
 #endif
diff --git a/sw/uiconfig/swriter/ui/numparapage.ui b/sw/uiconfig/swriter/ui/numparapage.ui
index 04b312d6f0bd..68a69e59284f 100644
--- a/sw/uiconfig/swriter/ui/numparapage.ui
+++ b/sw/uiconfig/swriter/ui/numparapage.ui
@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
+<!-- Generated with glade 3.22.1 -->
 <interface domain="sw">
-  <!-- interface-requires gtk+ 3.0 -->
+  <requires lib="gtk+" version="3.18"/>
   <object class="GtkAdjustment" id="adjustmentNF_RESTART_PARA">
     <property name="lower">1</property>
     <property name="upper">65000</property>
@@ -46,10 +47,10 @@
                     <property name="can_focus">False</property>
                     <property name="halign">start</property>
                     <property name="hexpand">True</property>
-                    <property name="xalign">0</property>
                     <property name="label" translatable="yes" context="numparapage|labelFT_OUTLINE_LEVEL">Outline level:</property>
                     <property name="use_underline">True</property>
                     <property name="mnemonic_widget">comboLB_OUTLINE_LEVEL</property>
+                    <property name="xalign">0</property>
                   </object>
                   <packing>
                     <property name="expand">False</property>
@@ -62,8 +63,6 @@
                     <property name="visible">True</property>
                     <property name="can_focus">False</property>
                     <property name="hexpand">True</property>
-                    <property name="entry_text_column">0</property>
-                    <property name="id_column">1</property>
                     <items>
                       <item translatable="yes" context="numparapage|comboLB_OUTLINE_LEVEL">Body text</item>
                       <item translatable="yes" context="numparapage|comboLB_OUTLINE_LEVEL">Level 1</item>
@@ -138,10 +137,10 @@
                         <property name="can_focus">False</property>
                         <property name="halign">start</property>
                         <property name="hexpand">True</property>
-                        <property name="xalign">0</property>
                         <property name="label" translatable="yes" context="numparapage|labelFT_NUMBER_STYLE">_Numbering style:</property>
                         <property name="use_underline">True</property>
                         <property name="mnemonic_widget">comboLB_NUMBER_STYLE</property>
+                        <property name="xalign">0</property>
                       </object>
                       <packing>
                         <property name="expand">False</property>
@@ -154,8 +153,6 @@
                         <property name="visible">True</property>
                         <property name="can_focus">False</property>
                         <property name="hexpand">True</property>
-                        <property name="entry_text_column">0</property>
-                        <property name="id_column">1</property>
                         <items>
                           <item translatable="yes" context="numparapage|comboLB_NUMBER_STYLE">None</item>
                         </items>
@@ -171,6 +168,7 @@
                         <property name="label" translatable="yes" context="numparapage|editnumstyle">Edit Style</property>
                         <property name="visible">True</property>
                         <property name="can_focus">False</property>
+                        <property name="receives_default">False</property>
                       </object>
                       <packing>
                         <property name="expand">False</property>
@@ -250,6 +248,7 @@
                         <property name="can_focus">True</property>
                         <property name="halign">start</property>
                         <property name="hexpand">True</property>
+                        <property name="activates_default">True</property>
                         <property name="adjustment">adjustmentSTART_WITH</property>
                       </object>
                       <packing>
@@ -362,10 +361,10 @@
                             <property name="can_focus">False</property>
                             <property name="halign">start</property>
                             <property name="margin_left">12</property>
-                            <property name="xalign">0</property>
                             <property name="label" translatable="yes" context="numparapage|labelFT_RESTART_NO">_Start with:</property>
                             <property name="use_underline">True</property>
                             <property name="mnemonic_widget">spinNF_RESTART_PARA</property>
+                            <property name="xalign">0</property>
                           </object>
                           <packing>
                             <property name="expand">True</property>
@@ -386,6 +385,7 @@
                         <property name="can_focus">True</property>
                         <property name="halign">start</property>
                         <property name="hexpand">True</property>
+                        <property name="activates_default">True</property>
                         <property name="adjustment">adjustmentNF_RESTART_PARA</property>
                       </object>
                       <packing>


More information about the Libreoffice-commits mailing list