comphelper/source/streaming/seqstream.cxx | 14 ++- i18npool/source/breakiterator/breakiteratorImpl.cxx | 2 include/comphelper/seqstream.hxx | 4 include/svx/dialogs.hrc | 3 svx/UIConfig_svx.mk | 1 svx/inc/helpid.hrc | 8 - svx/source/stbctrls/stbctrls.h | 10 -- svx/source/stbctrls/stbctrls.src | 57 ------------ svx/source/stbctrls/zoomctrl.cxx | 92 ++++++++++---------- svx/uiconfig/ui/zoommenu.ui | 73 +++++++++++++++ sw/inc/globals.hrc | 2 sw/source/uibase/utlui/navipi.src | 8 + 12 files changed, 147 insertions(+), 127 deletions(-)
New commits: commit 15c8a54cc9a875a17bed23a383fcd3d7742a1fca Author: Caolán McNamara <caol...@redhat.com> Date: Wed Apr 5 16:20:35 2017 +0100 convert zoom menu to .ui Change-Id: I4163042773f0f38e3372527d0a77a1b370bcafe9 diff --git a/include/svx/dialogs.hrc b/include/svx/dialogs.hrc index af5ee763eeab..5f98695ecace 100644 --- a/include/svx/dialogs.hrc +++ b/include/svx/dialogs.hrc @@ -140,9 +140,6 @@ #define RID_SVXFLOAT3D_FIX_G (RID_SVX_START + 78) #define RID_SVXFLOAT3D_FIX_B (RID_SVX_START + 84) -// ResId for zoom popup menu -#define RID_SVXMNU_ZOOM (RID_SVX_START + 82) - // ResId for funktion popup menue #define RID_SVXMNU_PSZ_FUNC (RID_SVX_START + 83) diff --git a/svx/UIConfig_svx.mk b/svx/UIConfig_svx.mk index ae2f419b73b1..e073234f3540 100644 --- a/svx/UIConfig_svx.mk +++ b/svx/UIConfig_svx.mk @@ -81,6 +81,7 @@ $(eval $(call gb_UIConfig_add_uifiles,svx,\ svx/uiconfig/ui/textunderlinecontrol \ svx/uiconfig/ui/xmlsecstatmenu \ svx/uiconfig/ui/xformspage \ + svx/uiconfig/ui/zoommenu \ )) # vim: set noet sw=4 ts=4: diff --git a/svx/inc/helpid.hrc b/svx/inc/helpid.hrc index 6ebd60d993ce..1844d102e9bb 100644 --- a/svx/inc/helpid.hrc +++ b/svx/inc/helpid.hrc @@ -53,14 +53,6 @@ #define HID_MNU_FUNC_NONE "SVX_HID_MNU_FUNC_NONE" #define HID_MNU_FUNC_SUM "SVX_HID_MNU_FUNC_SUM" #define HID_MNU_FUNC_SELECTION_COUNT "SVX_HID_MNU_FUNC_SELECTION_COUNT" -#define HID_MNU_ZOOM_100 "SVX_HID_MNU_ZOOM_100" -#define HID_MNU_ZOOM_150 "SVX_HID_MNU_ZOOM_150" -#define HID_MNU_ZOOM_200 "SVX_HID_MNU_ZOOM_200" -#define HID_MNU_ZOOM_50 "SVX_HID_MNU_ZOOM_50" -#define HID_MNU_ZOOM_75 "SVX_HID_MNU_ZOOM_75" -#define HID_MNU_ZOOM_OPTIMAL "SVX_HID_MNU_ZOOM_OPTIMAL" -#define HID_MNU_ZOOM_PAGE_WIDTH "SVX_HID_MNU_ZOOM_PAGE_WIDTH" -#define HID_MNU_ZOOM_WHOLE_PAGE "SVX_HID_MNU_ZOOM_WHOLE_PAGE" #define HID_POPUP_COLOR "SVX_HID_POPUP_COLOR" #define HID_POPUP_COLOR_CTRL "SVX_HID_POPUP_COLOR_CTRL" #define HID_POPUP_FRAME "SVX_HID_POPUP_FRAME" diff --git a/svx/source/stbctrls/stbctrls.h b/svx/source/stbctrls/stbctrls.h index 316cab0eb458..77d4f7f0f2fd 100644 --- a/svx/source/stbctrls/stbctrls.h +++ b/svx/source/stbctrls/stbctrls.h @@ -19,16 +19,6 @@ #ifndef INCLUDED_SVX_SOURCE_STBCTRLS_STBCTRLS_H #define INCLUDED_SVX_SOURCE_STBCTRLS_STBCTRLS_H -#define ZOOM_200 1 -#define ZOOM_150 2 -#define ZOOM_100 3 -#define ZOOM_75 4 -#define ZOOM_50 5 - -#define ZOOM_OPTIMAL 6 -#define ZOOM_PAGE_WIDTH 7 -#define ZOOM_WHOLE_PAGE 8 - // IDs as in SUBTOTAL_FUNC of Calc #define PSZ_FUNC_AVG 1 diff --git a/svx/source/stbctrls/stbctrls.src b/svx/source/stbctrls/stbctrls.src index e8f9c072e1b4..e846cffc5e14 100644 --- a/svx/source/stbctrls/stbctrls.src +++ b/svx/source/stbctrls/stbctrls.src @@ -188,62 +188,7 @@ String RID_SIDEBAR_EMPTY_PANEL_TEXT Text [ en-US ] = "Properties for the task that you are performing are not available for the current selection" ; }; - // PopupMenu ------------------------------------------------------------- -Menu RID_SVXMNU_ZOOM -{ - ItemList = - { - MenuItem - { - Identifier = ZOOM_WHOLE_PAGE ; - HelpId = HID_MNU_ZOOM_WHOLE_PAGE ; - Text [ en-US ] = "Entire Page" ; - }; - MenuItem - { - Identifier = ZOOM_PAGE_WIDTH ; - HelpId = HID_MNU_ZOOM_PAGE_WIDTH ; - Text [ en-US ] = "Page Width" ; - }; - MenuItem - { - Identifier = ZOOM_OPTIMAL ; - HelpId = HID_MNU_ZOOM_OPTIMAL ; - Text [ en-US ] = "Optimal View" ; - }; - MenuItem - { - Identifier = ZOOM_50 ; - HelpId = HID_MNU_ZOOM_50 ; - Text [ en-US ] = "50%" ; - }; - MenuItem - { - Identifier = ZOOM_75 ; - HelpId = HID_MNU_ZOOM_75 ; - Text [ en-US ] = "75%" ; - }; - MenuItem - { - Identifier = ZOOM_100 ; - HelpId = HID_MNU_ZOOM_100 ; - Text [ en-US ] = "100%" ; - }; - MenuItem - { - Identifier = ZOOM_150 ; - HelpId = HID_MNU_ZOOM_150 ; - Text [ en-US ] = "150%" ; - }; - MenuItem - { - Identifier = ZOOM_200 ; - HelpId = HID_MNU_ZOOM_200 ; - Text [ en-US ] = "200%" ; - }; - }; -}; - // Choice of functions on the SvxPosSizeStatusBarControl for Calc +// Choice of functions on the SvxPosSizeStatusBarControl for Calc Menu RID_SVXMNU_PSZ_FUNC { ItemList = diff --git a/svx/source/stbctrls/zoomctrl.cxx b/svx/source/stbctrls/zoomctrl.cxx index 384a0f9ed13e..e93740e9deb9 100644 --- a/svx/source/stbctrls/zoomctrl.cxx +++ b/svx/source/stbctrls/zoomctrl.cxx @@ -18,6 +18,7 @@ */ #include <i18nutil/unicode.hxx> +#include <vcl/builder.hxx> #include <vcl/svapp.hxx> #include <vcl/status.hxx> #include <vcl/menu.hxx> @@ -36,61 +37,63 @@ SFX_IMPL_STATUSBAR_CONTROL(SvxZoomStatusBarControl,SvxZoomItem); -class ZoomPopup_Impl : public PopupMenu +class ZoomPopup_Impl { public: ZoomPopup_Impl( sal_uInt16 nZ, SvxZoomEnableFlags nValueSet ); - sal_uInt16 GetZoom() const { return nZoom; } - sal_uInt16 GetCurId() const { return nCurId; } + sal_uInt16 GetZoom(); + OString GetCurItemIdent() const { return m_xMenu->GetCurItemIdent(); } + + sal_uInt16 Execute(vcl::Window* pWindow, const Point& rPopupPos) + { + return m_xMenu->Execute(pWindow, rPopupPos); + } private: + VclBuilder m_aBuilder; + VclPtr<PopupMenu> m_xMenu; sal_uInt16 nZoom; - sal_uInt16 nCurId; - - virtual void Select() override; }; - ZoomPopup_Impl::ZoomPopup_Impl( sal_uInt16 nZ, SvxZoomEnableFlags nValueSet ) - : PopupMenu(ResId(RID_SVXMNU_ZOOM, DIALOG_MGR())) + : m_aBuilder(nullptr, VclBuilderContainer::getUIRootDir(), "svx/ui/zoommenu.ui", "") + , m_xMenu(m_aBuilder.get_menu("menu")) , nZoom(nZ) - , nCurId(0) { if ( !(SvxZoomEnableFlags::N50 & nValueSet) ) - EnableItem( ZOOM_50, false ); + m_xMenu->EnableItem("50", false); if ( !(SvxZoomEnableFlags::N100 & nValueSet) ) - EnableItem( ZOOM_100, false ); + m_xMenu->EnableItem("100", false); if ( !(SvxZoomEnableFlags::N150 & nValueSet) ) - EnableItem( ZOOM_150, false ); + m_xMenu->EnableItem("150", false); if ( !(SvxZoomEnableFlags::N200 & nValueSet) ) - EnableItem( ZOOM_200, false ); + m_xMenu->EnableItem("200", false); if ( !(SvxZoomEnableFlags::OPTIMAL & nValueSet) ) - EnableItem( ZOOM_OPTIMAL, false ); + m_xMenu->EnableItem("optimal", false); if ( !(SvxZoomEnableFlags::WHOLEPAGE & nValueSet) ) - EnableItem( ZOOM_WHOLE_PAGE, false ); + m_xMenu->EnableItem("page", false); if ( !(SvxZoomEnableFlags::PAGEWIDTH & nValueSet) ) - EnableItem( ZOOM_PAGE_WIDTH, false ); + m_xMenu->EnableItem("width", false); } - -void ZoomPopup_Impl::Select() +sal_uInt16 ZoomPopup_Impl::GetZoom() { - nCurId = GetCurItemId(); - - switch ( nCurId ) - { - case ZOOM_200: nZoom = 200; break; - case ZOOM_150: nZoom = 150; break; - case ZOOM_100: nZoom = 100; break; - case ZOOM_75: nZoom = 75; break; - case ZOOM_50: nZoom = 50; break; - - case ZOOM_OPTIMAL: - case ZOOM_PAGE_WIDTH: - case ZOOM_WHOLE_PAGE: nZoom = 0; break; - - } + OString sIdent = GetCurItemIdent(); + if (sIdent == "200") + nZoom = 200; + else if (sIdent == "150") + nZoom = 150; + else if (sIdent == "100") + nZoom = 100; + else if (sIdent == "75") + nZoom = 75; + else if (sIdent == "50") + nZoom = 50; + else if (sIdent == "optimal" || sIdent == "width" || sIdent == "page") + nZoom = 0; + + return nZoom; } SvxZoomStatusBarControl::SvxZoomStatusBarControl( sal_uInt16 _nSlotId, @@ -142,20 +145,21 @@ void SvxZoomStatusBarControl::Command( const CommandEvent& rCEvt ) { if ( CommandEventId::ContextMenu == rCEvt.GetCommand() && bool(nValueSet) ) { - ScopedVclPtrInstance<ZoomPopup_Impl> aPop( nZoom, nValueSet ); + ZoomPopup_Impl aPop(nZoom, nValueSet); StatusBar& rStatusbar = GetStatusBar(); - if ( aPop->Execute( &rStatusbar, rCEvt.GetMousePosPixel() ) && ( nZoom != aPop->GetZoom() || !nZoom ) ) + if (aPop.Execute(&rStatusbar, rCEvt.GetMousePosPixel()) && (nZoom != aPop.GetZoom() || !nZoom)) { - nZoom = aPop->GetZoom(); - SvxZoomItem aZoom( SvxZoomType::PERCENT, nZoom, GetId() ); - - switch( aPop->GetCurId() ) - { - case ZOOM_OPTIMAL: aZoom.SetType( SvxZoomType::OPTIMAL ); break; - case ZOOM_PAGE_WIDTH: aZoom.SetType( SvxZoomType::PAGEWIDTH ); break; - case ZOOM_WHOLE_PAGE: aZoom.SetType( SvxZoomType::WHOLEPAGE ); break; - } + nZoom = aPop.GetZoom(); + SvxZoomItem aZoom(SvxZoomType::PERCENT, nZoom, GetId()); + + OString sIdent = aPop.GetCurItemIdent(); + if (sIdent == "optimal") + aZoom.SetType(SvxZoomType::OPTIMAL); + else if (sIdent == "width") + aZoom.SetType(SvxZoomType::PAGEWIDTH); + else if (sIdent == "page") + aZoom.SetType(SvxZoomType::WHOLEPAGE); css::uno::Any a; INetURLObject aObj( m_aCommandURL ); diff --git a/svx/uiconfig/ui/zoommenu.ui b/svx/uiconfig/ui/zoommenu.ui new file mode 100644 index 000000000000..3cc2c6c95beb --- /dev/null +++ b/svx/uiconfig/ui/zoommenu.ui @@ -0,0 +1,73 @@ +<?xml version="1.0" encoding="UTF-8"?> +<!-- Generated with glade 3.20.0 --> +<interface> + <requires lib="gtk+" version="3.10"/> + <object class="GtkMenu" id="menu"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <child> + <object class="GtkMenuItem" id="page"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="label" translatable="yes">Entire Page</property> + <property name="use_underline">True</property> + </object> + </child> + <child> + <object class="GtkMenuItem" id="width"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="label" translatable="yes">Page Width</property> + <property name="use_underline">True</property> + </object> + </child> + <child> + <object class="GtkMenuItem" id="optimal"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="label" translatable="yes">Optimal View</property> + <property name="use_underline">True</property> + </object> + </child> + <child> + <object class="GtkMenuItem" id="50"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="label" translatable="yes">50%</property> + <property name="use_underline">True</property> + </object> + </child> + <child> + <object class="GtkMenuItem" id="75"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="label" translatable="yes">75%</property> + <property name="use_underline">True</property> + </object> + </child> + <child> + <object class="GtkMenuItem" id="100"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="label" translatable="yes">100%</property> + <property name="use_underline">True</property> + </object> + </child> + <child> + <object class="GtkMenuItem" id="150"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="label" translatable="yes">150%</property> + <property name="use_underline">True</property> + </object> + </child> + <child> + <object class="GtkMenuItem" id="200"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="label" translatable="yes">200%</property> + <property name="use_underline">True</property> + </object> + </child> + </object> +</interface> commit e576278e16644a1478c9a90116ef4e383e54696c Author: Caolán McNamara <caol...@redhat.com> Date: Thu Apr 6 11:36:19 2017 +0100 reinstate removed but inuse resources since... commit 6742500ccb2ff6d39f527760152d2b08fd9cfe17 Date: Wed Mar 15 13:16:37 2017 +0200 remove unused defines from HRC files in sw/ Change-Id: Ie9cb54ba854ef04deaf9857468b3dbb171eb9b5a diff --git a/sw/inc/globals.hrc b/sw/inc/globals.hrc index 2eb78e46843e..748cd8b811c1 100644 --- a/sw/inc/globals.hrc +++ b/sw/inc/globals.hrc @@ -123,6 +123,8 @@ #define STR_OUTLINE_LEVEL (STR_CONTEXT_FIRST + 0) #define STR_DRAGMODE (STR_CONTEXT_FIRST + 1) #define STR_HYPERLINK (STR_CONTEXT_FIRST + 2) +#define STR_LINK_REGION (STR_CONTEXT_FIRST + 3) +#define STR_COPY_REGION (STR_CONTEXT_FIRST + 4) #define STR_DISPLAY (STR_CONTEXT_FIRST + 5) #define STR_ACTIVE_VIEW (STR_CONTEXT_FIRST + 6) #define STR_STATUS_FIRST (RC_GLOBALS_BEGIN + 59) diff --git a/sw/source/uibase/utlui/navipi.src b/sw/source/uibase/utlui/navipi.src index 0da7b7b8d157..f4473ef36442 100644 --- a/sw/source/uibase/utlui/navipi.src +++ b/sw/source/uibase/utlui/navipi.src @@ -118,6 +118,14 @@ String STR_HYPERLINK { Text [ en-US ] = "Insert as Hyperlink" ; }; +String STR_LINK_REGION +{ + Text [ en-US ] = "Insert as Link" ; +}; +String STR_COPY_REGION +{ + Text [ en-US ] = "Insert as Copy" ; +}; String STR_DISPLAY { Text [ en-US ] = "Display" ; commit 485242225388b5d6c559b56c654e816e91af34e7 Author: Caolán McNamara <caol...@redhat.com> Date: Wed Apr 5 10:04:13 2017 +0100 split closeOutput up Change-Id: Id6408fa804e1049ceab012ec0220d8e6f8d8e555 diff --git a/comphelper/source/streaming/seqstream.cxx b/comphelper/source/streaming/seqstream.cxx index b375f7c6d6be..f1e397d13770 100644 --- a/comphelper/source/streaming/seqstream.cxx +++ b/comphelper/source/streaming/seqstream.cxx @@ -199,12 +199,9 @@ void SAL_CALL OSequenceOutputStream::flush( ) m_rSequence.realloc(m_nSize); } - -void SAL_CALL OSequenceOutputStream::closeOutput( ) +void OSequenceOutputStream::finalizeOutput() { MutexGuard aGuard(m_aMutex); - if (!m_bConnected) - throw NotConnectedException(); // cut the sequence to the real size m_rSequence.realloc(m_nSize); @@ -212,6 +209,15 @@ void SAL_CALL OSequenceOutputStream::closeOutput( ) m_bConnected = false; } +void SAL_CALL OSequenceOutputStream::closeOutput() +{ + MutexGuard aGuard(m_aMutex); + if (!m_bConnected) + throw NotConnectedException(); + + finalizeOutput(); +} + } // namespace comphelper /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/include/comphelper/seqstream.hxx b/include/comphelper/seqstream.hxx index 794a56837c78..ae7502f776e4 100644 --- a/include/comphelper/seqstream.hxx +++ b/include/comphelper/seqstream.hxx @@ -71,6 +71,8 @@ class SAL_DLLPUBLIC_TEMPLATE OSequenceOutputStream_Base class COMPHELPER_DLLPUBLIC OSequenceOutputStream : public OSequenceOutputStream_Base { +private: + void finalizeOutput(); protected: css::uno::Sequence< sal_Int8 >& m_rSequence; double m_nResizeFactor; @@ -85,7 +87,7 @@ protected: ::osl::Mutex m_aMutex; protected: - virtual ~OSequenceOutputStream() override { if (m_bConnected) closeOutput(); } + virtual ~OSequenceOutputStream() override { if (m_bConnected) finalizeOutput(); } public: /** constructs the object. Everything written into the stream through the XOutputStream methods will be forwarded commit 65753b9713ae798804da078dd4e9ed3e478e5264 Author: Caolán McNamara <caol...@redhat.com> Date: Thu Apr 6 09:29:39 2017 +0100 scriptclass for codepoint 0 is WEAK getScriptClass returns the cached nRet if new char equals old char old char is initialized to 0, so nRet should be initialized to the script class 0 would map to, which is WEAK (1), not 0 Change-Id: I30a8388d7c91e786c88deee781a7e2f80f219207 diff --git a/i18npool/source/breakiterator/breakiteratorImpl.cxx b/i18npool/source/breakiterator/breakiteratorImpl.cxx index 25e3f367ac98..2c4971562a7b 100644 --- a/i18npool/source/breakiterator/breakiteratorImpl.cxx +++ b/i18npool/source/breakiterator/breakiteratorImpl.cxx @@ -509,7 +509,7 @@ bool getCompatibilityScriptClassByBlock(sal_uInt32 currentChar, sal_Int16 &rScri sal_Int16 BreakIteratorImpl::getScriptClass(sal_uInt32 currentChar) { static sal_uInt32 lastChar = 0; - static sal_Int16 nRet = 0; + static sal_Int16 nRet = ScriptType::WEAK; if (currentChar != lastChar) {
_______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits