diff options
author | Caolán McNamara <caolanm@redhat.com> | 2018-06-18 11:10:10 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2018-06-18 22:05:23 +0200 |
commit | 935fd2d2b776003a89d441bcd243fd079cc0fa65 (patch) | |
tree | 926807dc6f391cf3cf8ead877869e92a52cdf290 /filter | |
parent | f8b74d8095bfea66aa47a0d70839807478989396 (diff) |
weld ImpPDFTabLinksPage
Change-Id: I6e10b135d102f56d42a9f456557150cb557dd788
Reviewed-on: https://gerrit.libreoffice.org/56037
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'filter')
-rw-r--r-- | filter/source/pdf/impdialog.cxx | 114 | ||||
-rw-r--r-- | filter/source/pdf/impdialog.hxx | 23 | ||||
-rw-r--r-- | filter/uiconfig/ui/pdflinkspage.ui | 18 |
3 files changed, 59 insertions, 96 deletions
diff --git a/filter/source/pdf/impdialog.cxx b/filter/source/pdf/impdialog.cxx index e48641bef88a..7474ba9a648f 100644 --- a/filter/source/pdf/impdialog.cxx +++ b/filter/source/pdf/impdialog.cxx @@ -1365,53 +1365,33 @@ void ImpPDFTabSecurityPage::ImplPDFASecurityControl( bool bEnableSecurity ) enablePermissionControls(); } - /// The link preferences tab page (relative and other stuff) -ImpPDFTabLinksPage::ImpPDFTabLinksPage( vcl::Window* pParent, - const SfxItemSet& rCoreSet ) : - SfxTabPage( pParent, "PdfLinksPage","filter/ui/pdflinkspage.ui",&rCoreSet ), - - mbOpnLnksDefaultUserState( false ), - mbOpnLnksLaunchUserState( false ), - mbOpnLnksBrowserUserState( false ) +ImpPDFTabLinksPage::ImpPDFTabLinksPage(TabPageParent pParent, const SfxItemSet& rCoreSet) + : SfxTabPage(pParent, "filter/ui/pdflinkspage.ui", "PdfLinksPage", &rCoreSet) + , mbOpnLnksDefaultUserState(false) + , mbOpnLnksLaunchUserState(false) + , mbOpnLnksBrowserUserState(false) + , m_xCbExprtBmkrToNmDst(m_xBuilder->weld_check_button("export")) + , m_xCbOOoToPDFTargets(m_xBuilder->weld_check_button("convert")) + , m_xCbExportRelativeFsysLinks(m_xBuilder->weld_check_button("exporturl")) + , m_xRbOpnLnksDefault(m_xBuilder->weld_radio_button("default")) + , m_xRbOpnLnksLaunch(m_xBuilder->weld_radio_button("openpdf")) + , m_xRbOpnLnksBrowser(m_xBuilder->weld_radio_button("openinternet")) { - get(m_pCbExprtBmkrToNmDst,"export"); - get(m_pCbOOoToPDFTargets ,"convert"); - get(m_pCbExportRelativeFsysLinks ,"exporturl"); - get(m_pRbOpnLnksDefault ,"default"); - get(m_pRbOpnLnksLaunch ,"openpdf"); - get(m_pRbOpnLnksBrowser ,"openinternet"); } - ImpPDFTabLinksPage::~ImpPDFTabLinksPage() { - disposeOnce(); } - -void ImpPDFTabLinksPage::dispose() +VclPtr<SfxTabPage> ImpPDFTabLinksPage::Create(TabPageParent pParent, const SfxItemSet* rAttrSet) { - m_pCbExprtBmkrToNmDst.clear(); - m_pCbOOoToPDFTargets.clear(); - m_pCbExportRelativeFsysLinks.clear(); - m_pRbOpnLnksDefault.clear(); - m_pRbOpnLnksLaunch.clear(); - m_pRbOpnLnksBrowser.clear(); - SfxTabPage::dispose(); + return VclPtr<ImpPDFTabLinksPage>::Create(pParent, *rAttrSet); } - -VclPtr<SfxTabPage> ImpPDFTabLinksPage::Create( TabPageParent pParent, - const SfxItemSet* rAttrSet) -{ - return VclPtr<ImpPDFTabLinksPage>::Create( pParent.pParent, *rAttrSet ); -} - - void ImpPDFTabLinksPage::GetFilterConfigItem( ImpPDFTabDialog* paParent ) { - paParent->mbExportRelativeFsysLinks = m_pCbExportRelativeFsysLinks->IsChecked(); + paParent->mbExportRelativeFsysLinks = m_xCbExportRelativeFsysLinks->get_active(); bool bIsPDFASel = false; ImpPDFTabGeneralPage* pGeneralPage = paParent->getGeneralPage(); @@ -1421,9 +1401,9 @@ void ImpPDFTabLinksPage::GetFilterConfigItem( ImpPDFTabDialog* paParent ) if( !bIsPDFASel ) { // ...get the control states - mbOpnLnksDefaultUserState = m_pRbOpnLnksDefault->IsChecked(); - mbOpnLnksLaunchUserState = m_pRbOpnLnksLaunch->IsChecked(); - mbOpnLnksBrowserUserState = m_pRbOpnLnksBrowser->IsChecked(); + mbOpnLnksDefaultUserState = m_xRbOpnLnksDefault->get_active(); + mbOpnLnksLaunchUserState = m_xRbOpnLnksLaunch->get_active(); + mbOpnLnksBrowserUserState = m_xRbOpnLnksBrowser->get_active(); } // the control states, or the saved is used // to form the stored selection @@ -1433,33 +1413,32 @@ void ImpPDFTabLinksPage::GetFilterConfigItem( ImpPDFTabDialog* paParent ) else if( mbOpnLnksLaunchUserState ) paParent->mnViewPDFMode = 1; - paParent->mbConvertOOoTargets = m_pCbOOoToPDFTargets->IsChecked(); - paParent->mbExportBmkToPDFDestination = m_pCbExprtBmkrToNmDst->IsChecked(); + paParent->mbConvertOOoTargets = m_xCbOOoToPDFTargets->get_active(); + paParent->mbExportBmkToPDFDestination = m_xCbExprtBmkrToNmDst->get_active(); } - void ImpPDFTabLinksPage::SetFilterConfigItem( const ImpPDFTabDialog* paParent ) { - m_pCbOOoToPDFTargets->Check( paParent->mbConvertOOoTargets ); - m_pCbExprtBmkrToNmDst->Check( paParent->mbExportBmkToPDFDestination ); + m_xCbOOoToPDFTargets->set_active(paParent->mbConvertOOoTargets); + m_xCbExprtBmkrToNmDst->set_active(paParent->mbExportBmkToPDFDestination); - m_pRbOpnLnksDefault->SetClickHdl( LINK( this, ImpPDFTabLinksPage, ClickRbOpnLnksDefaultHdl ) ); - m_pRbOpnLnksBrowser->SetClickHdl( LINK( this, ImpPDFTabLinksPage, ClickRbOpnLnksBrowserHdl ) ); + m_xRbOpnLnksDefault->connect_toggled(LINK(this, ImpPDFTabLinksPage, ClickRbOpnLnksDefaultHdl)); + m_xRbOpnLnksBrowser->connect_toggled(LINK(this, ImpPDFTabLinksPage, ClickRbOpnLnksBrowserHdl)); - m_pCbExportRelativeFsysLinks->Check( paParent->mbExportRelativeFsysLinks ); + m_xCbExportRelativeFsysLinks->set_active(paParent->mbExportRelativeFsysLinks); switch( paParent->mnViewPDFMode ) { default: case 0: - m_pRbOpnLnksDefault->Check(); + m_xRbOpnLnksDefault->set_active(true); mbOpnLnksDefaultUserState = true; break; case 1: - m_pRbOpnLnksLaunch->Check(); + m_xRbOpnLnksLaunch->set_active(true); mbOpnLnksLaunchUserState = true; break; case 2: - m_pRbOpnLnksBrowser->Check(); + m_xRbOpnLnksBrowser->set_active(true); mbOpnLnksBrowserUserState = true; break; } @@ -1482,43 +1461,40 @@ void ImpPDFTabLinksPage::ImplPDFALinkControl( bool bEnableLaunch ) // set the value and position of link type selection if( bEnableLaunch ) { - m_pRbOpnLnksLaunch->Enable(); + m_xRbOpnLnksLaunch->set_sensitive(true); // restore user state with no PDF/A-1 selected - m_pRbOpnLnksDefault->Check( mbOpnLnksDefaultUserState ); - m_pRbOpnLnksLaunch->Check( mbOpnLnksLaunchUserState ); - m_pRbOpnLnksBrowser->Check( mbOpnLnksBrowserUserState ); + m_xRbOpnLnksDefault->set_active(mbOpnLnksDefaultUserState); + m_xRbOpnLnksLaunch->set_active(mbOpnLnksLaunchUserState); + m_xRbOpnLnksBrowser->set_active(mbOpnLnksBrowserUserState); } else { // save user state with no PDF/A-1 selected - mbOpnLnksDefaultUserState = m_pRbOpnLnksDefault->IsChecked(); - mbOpnLnksLaunchUserState = m_pRbOpnLnksLaunch->IsChecked(); - mbOpnLnksBrowserUserState = m_pRbOpnLnksBrowser->IsChecked(); - m_pRbOpnLnksLaunch->Enable( false ); - if( mbOpnLnksLaunchUserState ) - m_pRbOpnLnksBrowser->Check(); + mbOpnLnksDefaultUserState = m_xRbOpnLnksDefault->get_active(); + mbOpnLnksLaunchUserState = m_xRbOpnLnksLaunch->get_active(); + mbOpnLnksBrowserUserState = m_xRbOpnLnksBrowser->get_active(); + m_xRbOpnLnksLaunch->set_sensitive(false); + if (mbOpnLnksLaunchUserState) + m_xRbOpnLnksBrowser->set_active(true); } } - /// Reset the memory of Launch action present when PDF/A-1 was requested -IMPL_LINK_NOARG(ImpPDFTabLinksPage, ClickRbOpnLnksDefaultHdl, Button*, void) +IMPL_LINK_NOARG(ImpPDFTabLinksPage, ClickRbOpnLnksDefaultHdl, weld::ToggleButton&, void) { - mbOpnLnksDefaultUserState = m_pRbOpnLnksDefault->IsChecked(); - mbOpnLnksLaunchUserState = m_pRbOpnLnksLaunch->IsChecked(); - mbOpnLnksBrowserUserState = m_pRbOpnLnksBrowser->IsChecked(); + mbOpnLnksDefaultUserState = m_xRbOpnLnksDefault->get_active(); + mbOpnLnksLaunchUserState = m_xRbOpnLnksLaunch->get_active(); + mbOpnLnksBrowserUserState = m_xRbOpnLnksBrowser->get_active(); } - /// Reset the memory of a launch action present when PDF/A-1 was requested -IMPL_LINK_NOARG(ImpPDFTabLinksPage, ClickRbOpnLnksBrowserHdl, Button*, void) +IMPL_LINK_NOARG(ImpPDFTabLinksPage, ClickRbOpnLnksBrowserHdl, weld::ToggleButton&, void) { - mbOpnLnksDefaultUserState = m_pRbOpnLnksDefault->IsChecked(); - mbOpnLnksLaunchUserState = m_pRbOpnLnksLaunch->IsChecked(); - mbOpnLnksBrowserUserState = m_pRbOpnLnksBrowser->IsChecked(); + mbOpnLnksDefaultUserState = m_xRbOpnLnksDefault->get_active(); + mbOpnLnksLaunchUserState = m_xRbOpnLnksLaunch->get_active(); + mbOpnLnksBrowserUserState = m_xRbOpnLnksBrowser->get_active(); } - ImplErrorDialog::ImplErrorDialog(weld::Window* pParent, const std::set<vcl::PDFWriter::ErrorCode>& rErrors) : MessageDialogController(pParent, "filter/ui/warnpdfdialog.ui", "WarnPDFDialog", "grid") , m_xErrors(m_xBuilder->weld_tree_view("errors")) diff --git a/filter/source/pdf/impdialog.hxx b/filter/source/pdf/impdialog.hxx index ddd86c8a8473..f4613f0757d3 100644 --- a/filter/source/pdf/impdialog.hxx +++ b/filter/source/pdf/impdialog.hxx @@ -376,26 +376,25 @@ public: /// Implements the relative link stuff class ImpPDFTabLinksPage : public SfxTabPage { - VclPtr<CheckBox> m_pCbExprtBmkrToNmDst; - VclPtr<CheckBox> m_pCbOOoToPDFTargets; - VclPtr<CheckBox> m_pCbExportRelativeFsysLinks; - - VclPtr<RadioButton> m_pRbOpnLnksDefault; bool mbOpnLnksDefaultUserState; - VclPtr<RadioButton> m_pRbOpnLnksLaunch; bool mbOpnLnksLaunchUserState; - VclPtr<RadioButton> m_pRbOpnLnksBrowser; bool mbOpnLnksBrowserUserState; - DECL_LINK( ClickRbOpnLnksDefaultHdl, Button*, void ); - DECL_LINK( ClickRbOpnLnksBrowserHdl, Button*, void ); + std::unique_ptr<weld::CheckButton> m_xCbExprtBmkrToNmDst; + std::unique_ptr<weld::CheckButton> m_xCbOOoToPDFTargets; + std::unique_ptr<weld::CheckButton> m_xCbExportRelativeFsysLinks; + std::unique_ptr<weld::RadioButton> m_xRbOpnLnksDefault; + std::unique_ptr<weld::RadioButton> m_xRbOpnLnksLaunch; + std::unique_ptr<weld::RadioButton> m_xRbOpnLnksBrowser; + + DECL_LINK(ClickRbOpnLnksDefaultHdl, weld::ToggleButton&, void); + DECL_LINK(ClickRbOpnLnksBrowserHdl, weld::ToggleButton&, void); public: - ImpPDFTabLinksPage( vcl::Window* pParent, const SfxItemSet& rSet ); + ImpPDFTabLinksPage(TabPageParent pParent, const SfxItemSet& rSet); virtual ~ImpPDFTabLinksPage() override; - virtual void dispose() override; - static VclPtr<SfxTabPage> Create( TabPageParent pParent, const SfxItemSet* rAttrSet ); + static VclPtr<SfxTabPage> Create(TabPageParent pParent, const SfxItemSet* rAttrSet); void GetFilterConfigItem( ImpPDFTabDialog* paParent); void SetFilterConfigItem( const ImpPDFTabDialog* paParent ); diff --git a/filter/uiconfig/ui/pdflinkspage.ui b/filter/uiconfig/ui/pdflinkspage.ui index ba3562afc3df..656130852f0e 100644 --- a/filter/uiconfig/ui/pdflinkspage.ui +++ b/filter/uiconfig/ui/pdflinkspage.ui @@ -1,6 +1,7 @@ <?xml version="1.0" encoding="UTF-8"?> +<!-- Generated with glade 3.22.1 --> <interface domain="flt"> - <!-- interface-requires gtk+ 3.0 --> + <requires lib="gtk+" version="3.0"/> <object class="GtkBox" id="PdfLinksPage"> <property name="visible">True</property> <property name="can_focus">False</property> @@ -38,8 +39,6 @@ <packing> <property name="left_attach">0</property> <property name="top_attach">0</property> - <property name="width">1</property> - <property name="height">1</property> </packing> </child> <child> @@ -56,8 +55,6 @@ <packing> <property name="left_attach">0</property> <property name="top_attach">1</property> - <property name="width">1</property> - <property name="height">1</property> </packing> </child> <child> @@ -74,8 +71,6 @@ <packing> <property name="left_attach">0</property> <property name="top_attach">2</property> - <property name="width">1</property> - <property name="height">1</property> </packing> </child> </object> @@ -126,13 +121,10 @@ <property name="xalign">0</property> <property name="active">True</property> <property name="draw_indicator">True</property> - <property name="group">openpdf</property> </object> <packing> <property name="left_attach">0</property> <property name="top_attach">0</property> - <property name="width">1</property> - <property name="height">1</property> </packing> </child> <child> @@ -144,13 +136,11 @@ <property name="use_underline">True</property> <property name="xalign">0</property> <property name="draw_indicator">True</property> - <property name="group">openinternet</property> + <property name="group">default</property> </object> <packing> <property name="left_attach">0</property> <property name="top_attach">1</property> - <property name="width">1</property> - <property name="height">1</property> </packing> </child> <child> @@ -167,8 +157,6 @@ <packing> <property name="left_attach">0</property> <property name="top_attach">2</property> - <property name="width">1</property> - <property name="height">1</property> </packing> </child> </object> |