weld ScNumberFormat ItemWindow

Change-Id: Ib019912e91be617edbbcdf2fafb92d6685487b98
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88174
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>

3 files changed, 58 insertions, 27 deletions

diff --git a/sc/source/ui/cctrl/cbnumberformat.cxx b/sc/source/ui/cctrl/cbnumberformat.cxx
index 2e08b0256378..830e24761d08 100644
--- a/sc/source/ui/cctrl/cbnumberformat.cxx
+++ b/sc/source/ui/cctrl/cbnumberformat.cxx
@@ -22,40 +22,63 @@
 #include <scresid.hxx>
 #include <sfx2/dispatch.hxx>
 #include <sfx2/viewfrm.hxx>
+#include <sfx2/viewsh.hxx>
 #include <svl/intitem.hxx>
 #include <sc.hrc>
 
-ScNumberFormat::ScNumberFormat(vcl::Window* pParent, WinBits nStyle) :
-    ListBox(pParent, nStyle | WB_DROPDOWN|WB_LEFT|WB_VCENTER|WB_BORDER|WB_3DLOOK)
+ScNumberFormat::ScNumberFormat(vcl::Window* pParent)
+    : InterimItemWindow(pParent, "modules/scalc/ui/numberbox.ui", "NumberBox")
+    , m_xWidget(m_xBuilder->weld_combo_box("numbertype"))
 {
-    SetSelectHdl(LINK(this, ScNumberFormat, NumFormatSelectHdl));
-    AdaptDropDownLineCountToMaximum();
-
-    InsertEntry(ScResId(STR_GENERAL));
-    InsertEntry(ScResId(STR_NUMBER));
-    InsertEntry(ScResId(STR_PERCENT));
-    InsertEntry(ScResId(STR_CURRENCY));
-    InsertEntry(ScResId(STR_DATE));
-    InsertEntry(ScResId(STR_TIME));
-    InsertEntry(ScResId(STR_SCIENTIFIC));
-    InsertEntry(ScResId(STR_FRACTION));
-    InsertEntry(ScResId(STR_BOOLEAN_VALUE));
-    InsertEntry(ScResId(STR_TEXT));
+    m_xWidget->append_text(ScResId(STR_GENERAL));
+    m_xWidget->append_text(ScResId(STR_NUMBER));
+    m_xWidget->append_text(ScResId(STR_PERCENT));
+    m_xWidget->append_text(ScResId(STR_CURRENCY));
+    m_xWidget->append_text(ScResId(STR_DATE));
+    m_xWidget->append_text(ScResId(STR_TIME));
+    m_xWidget->append_text(ScResId(STR_SCIENTIFIC));
+    m_xWidget->append_text(ScResId(STR_FRACTION));
+    m_xWidget->append_text(ScResId(STR_BOOLEAN_VALUE));
+    m_xWidget->append_text(ScResId(STR_TEXT));
+
+    m_xWidget->connect_changed(LINK(this, ScNumberFormat, NumFormatSelectHdl));
+    m_xWidget->connect_key_press(LINK(this, ScNumberFormat, KeyInputHdl));
+
+    SetSizePixel(m_xWidget->get_preferred_size());
+}
+
+void ScNumberFormat::dispose()
+{
+    m_xWidget.reset();
+    InterimItemWindow::dispose();
+}
+
+ScNumberFormat::~ScNumberFormat()
+{
+    disposeOnce();
 }
 
-IMPL_STATIC_LINK(ScNumberFormat, NumFormatSelectHdl, ListBox&, rBox, void)
+IMPL_STATIC_LINK(ScNumberFormat, NumFormatSelectHdl, weld::ComboBox&, rBox, void)
 {
-    if(SfxViewFrame::Current())
+    auto* pCurSh = SfxViewFrame::Current();
+    if (pCurSh)
     {
-        SfxDispatcher* pDisp = SfxViewFrame::Current()->GetBindings().GetDispatcher();
-        if(pDisp)
+        SfxDispatcher* pDisp = pCurSh->GetBindings().GetDispatcher();
+        if (pDisp)
         {
-            const sal_Int32 nVal = rBox.GetSelectedEntryPos();
+            const sal_Int32 nVal = rBox.get_active();
             SfxUInt16Item aItem(SID_NUMBER_TYPE_FORMAT, nVal);
             pDisp->ExecuteList(SID_NUMBER_TYPE_FORMAT,
                     SfxCallMode::RECORD, {&aItem});
+
+            pCurSh->GetWindow().GrabFocus();
         }
     }
 }
 
+IMPL_LINK(ScNumberFormat, KeyInputHdl, const KeyEvent&, rKEvt, bool)
+{
+    return ChildKeyInput(rKEvt);
+}
+
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sc/source/ui/inc/cbnumberformat.hxx b/sc/source/ui/inc/cbnumberformat.hxx
index 9773fd9b34cc..c2e2f40366bc 100644
--- a/sc/source/ui/inc/cbnumberformat.hxx
+++ b/sc/source/ui/inc/cbnumberformat.hxx
@@ -20,15 +20,22 @@
 #ifndef INCLUDED_SC_SOURCE_UI_INC_CBNUMBERFORMAT_HXX
 #define INCLUDED_SC_SOURCE_UI_INC_CBNUMBERFORMAT_HXX
 
-#include <vcl/lstbox.hxx>
+#include <sfx2/InterimItemWindow.hxx>
 
-class ScNumberFormat : public ListBox
+class ScNumberFormat final : public InterimItemWindow
 {
 public:
-    explicit ScNumberFormat(vcl::Window* pParent, WinBits nStyle);
+    explicit ScNumberFormat(vcl::Window* pParent);
+    virtual void dispose() override;
+    virtual ~ScNumberFormat() override;
+
+    void set_active(int nPos) { m_xWidget->set_active(nPos); }
 
 private:
-    DECL_STATIC_LINK(ScNumberFormat, NumFormatSelectHdl, ListBox&, void);
+    std::unique_ptr<weld::ComboBox> m_xWidget;
+
+    DECL_STATIC_LINK(ScNumberFormat, NumFormatSelectHdl, weld::ComboBox&, void);
+    DECL_LINK(KeyInputHdl, const KeyEvent&, bool);
 };
 
 #endif
diff --git a/sc/source/ui/sidebar/NumberFormatControl.cxx b/sc/source/ui/sidebar/NumberFormatControl.cxx
index 9e05fcc5a545..dbfc7debf8b5 100644
--- a/sc/source/ui/sidebar/NumberFormatControl.cxx
+++ b/sc/source/ui/sidebar/NumberFormatControl.cxx
@@ -57,7 +57,7 @@ void ScNumberFormatControl::StateChanged(sal_uInt16, SfxItemState eState,
         {
             const SfxUInt16Item* pItem = static_cast<const SfxUInt16Item*>(pState);
             sal_uInt16 nVal = pItem->GetValue();
-            pComboBox->SelectEntryPos(nVal);
+            pComboBox->set_active(nVal);
             break;
         }
 
@@ -68,8 +68,9 @@ void ScNumberFormatControl::StateChanged(sal_uInt16, SfxItemState eState,
 
 VclPtr<vcl::Window> ScNumberFormatControl::CreateItemWindow( vcl::Window *pParent )
 {
-    VclPtr<ScNumberFormat> pControl = VclPtr<ScNumberFormat>::Create(pParent, WB_DROPDOWN);
-    pControl->SetSizePixel(pControl->GetOptimalSize());
+    VclPtr<ScNumberFormat> pControl = VclPtr<ScNumberFormat>::Create(pParent);
+    pControl->Show();
+
     return pControl;
 }