include/sfx2/printopt.hxx | 1 - include/svtools/restartdialog.hxx | 3 --- sfx2/source/dialog/printopt.cxx | 23 +---------------------- sfx2/uiconfig/ui/optprintpage.ui | 20 +++----------------- svtools/source/dialogs/restartdialog.cxx | 3 --- svtools/uiconfig/ui/restartdialog.ui | 17 +---------------- 6 files changed, 5 insertions(+), 62 deletions(-)
New commits: commit 2a40505d00fcd79bfa433c301ddc05a35287bfb1 Author: Caolán McNamara <caol...@redhat.com> AuthorDate: Thu Jan 26 11:49:43 2023 +0000 Commit: Caolán McNamara <caol...@redhat.com> CommitDate: Thu Jan 26 15:09:20 2023 +0000 remove option to toggle IsPDFAsStandardPrintJobFormat from UI to deprecate it. Still possible to toggle from expert config for now, but we should push on and remove this and all the postscript printing at this point. Change-Id: If8869a62616b336764720f5dc56d86b94f16ffbe Reviewed-on: https://gerrit.libreoffice.org/c/core/+/146169 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caol...@redhat.com> diff --git a/include/sfx2/printopt.hxx b/include/sfx2/printopt.hxx index ef7fd4683a7d..2cc4b0a1ded6 100644 --- a/include/sfx2/printopt.hxx +++ b/include/sfx2/printopt.hxx @@ -46,7 +46,6 @@ private: std::unique_ptr<weld::ComboBox> m_xReduceBitmapsResolutionLB; std::unique_ptr<weld::CheckButton> m_xReduceBitmapsTransparencyCB; std::unique_ptr<weld::CheckButton> m_xConvertToGreyscalesCB; - std::unique_ptr<weld::CheckButton> m_xPDFCB; std::unique_ptr<weld::CheckButton> m_xPaperSizeCB; std::unique_ptr<weld::CheckButton> m_xPaperOrientationCB; std::unique_ptr<weld::CheckButton> m_xTransparencyCB; diff --git a/include/svtools/restartdialog.hxx b/include/svtools/restartdialog.hxx index 001f322772a5..89178ad26e36 100644 --- a/include/svtools/restartdialog.hxx +++ b/include/svtools/restartdialog.hxx @@ -26,9 +26,6 @@ enum RestartReason { // "For the selected Java runtime environment to work properly, // %PRODUCTNAME must be restarted.": RESTART_REASON_JAVA, - // For the modified default print job format to take effect, - // %PRODUCTNAME must be restarted: - RESTART_REASON_PDF_AS_STANDARD_JOB_FORMAT, // "For the bibliography to work properly, // %PRODUCTNAME must be restarted.": RESTART_REASON_BIBLIOGRAPHY_INSTALL, diff --git a/sfx2/source/dialog/printopt.cxx b/sfx2/source/dialog/printopt.cxx index 8d16edfc0e3e..d804bdf8f202 100644 --- a/sfx2/source/dialog/printopt.cxx +++ b/sfx2/source/dialog/printopt.cxx @@ -50,24 +50,14 @@ SfxCommonPrintOptionsTabPage::SfxCommonPrintOptionsTabPage(weld::Container* pPag , m_xReduceBitmapsResolutionLB(m_xBuilder->weld_combo_box("reducebitmapdpi")) , m_xReduceBitmapsTransparencyCB(m_xBuilder->weld_check_button("reducebitmaptrans")) , m_xConvertToGreyscalesCB(m_xBuilder->weld_check_button("converttogray")) - , m_xPDFCB(m_xBuilder->weld_check_button("pdf")) , m_xPaperSizeCB(m_xBuilder->weld_check_button("papersize")) , m_xPaperOrientationCB(m_xBuilder->weld_check_button("paperorient")) , m_xTransparencyCB(m_xBuilder->weld_check_button("trans")) { -#ifndef ENABLE_CUPS - m_xPDFCB->hide(); -#endif - - if( bOutputForPrinter ) - { + if (bOutputForPrinter) m_xPrinterOutputRB->set_active(true); - } else - { m_xPrintFileOutputRB->set_active(true); - m_xPDFCB->set_sensitive(false); - } m_xPrinterOutputRB->connect_toggled( LINK( this, SfxCommonPrintOptionsTabPage, ToggleOutputPrinterRBHdl ) ); m_xPrintFileOutputRB->connect_toggled( LINK( this, SfxCommonPrintOptionsTabPage, ToggleOutputPrintFileRBHdl ) ); @@ -182,7 +172,6 @@ void SfxCommonPrintOptionsTabPage::ImplUpdateControls( const vcl::printer::Optio m_xReduceBitmapsTransparencyCB->set_active( pCurrentOptions->IsReducedBitmapIncludesTransparency() ); m_xConvertToGreyscalesCB->set_active( pCurrentOptions->IsConvertToGreyscales() ); - m_xPDFCB->set_active( pCurrentOptions->IsPDFAsStandardPrintJobFormat() ); ClickReduceTransparencyCBHdl(*m_xReduceTransparencyCB); ClickReduceGradientsCBHdl(*m_xReduceGradientsCB); @@ -203,14 +192,6 @@ void SfxCommonPrintOptionsTabPage::ImplSaveControls( vcl::printer::Options* pCur SAL_N_ELEMENTS(aDPIArray) - 1 ) ] ); pCurrentOptions->SetReducedBitmapIncludesTransparency( m_xReduceBitmapsTransparencyCB->get_active() ); pCurrentOptions->SetConvertToGreyscales( m_xConvertToGreyscalesCB->get_active() ); - bool bOrigBackEnd = pCurrentOptions->IsPDFAsStandardPrintJobFormat(); - if (bOrigBackEnd != m_xPDFCB->get_active()) - { - pCurrentOptions->SetPDFAsStandardPrintJobFormat( m_xPDFCB->get_active() ); - svtools::executeRestartDialog( - comphelper::getProcessComponentContext(), nullptr, - svtools::RESTART_REASON_PDF_AS_STANDARD_JOB_FORMAT); - } } IMPL_LINK_NOARG( SfxCommonPrintOptionsTabPage, ClickReduceTransparencyCBHdl, weld::Toggleable&, void ) @@ -278,12 +259,10 @@ IMPL_LINK( SfxCommonPrintOptionsTabPage, ToggleOutputPrintFileRBHdl, weld::Toggl { ImplUpdateControls( &maPrintFileOptions ); bOutputForPrinter = false; - m_xPDFCB->set_sensitive(false); } else { ImplSaveControls( &maPrintFileOptions ); - m_xPDFCB->set_sensitive(true); } } diff --git a/sfx2/uiconfig/ui/optprintpage.ui b/sfx2/uiconfig/ui/optprintpage.ui index 4af9bf58406c..615b818e7152 100644 --- a/sfx2/uiconfig/ui/optprintpage.ui +++ b/sfx2/uiconfig/ui/optprintpage.ui @@ -588,27 +588,13 @@ <property name="label-xalign">0</property> <property name="shadow-type">none</property> <child> - <!-- n-columns=1 n-rows=3 --> + <!-- n-columns=1 n-rows=2 --> <object class="GtkGrid" id="grid7"> <property name="visible">True</property> <property name="can-focus">False</property> <property name="margin-start">12</property> <property name="margin-top">6</property> <property name="row-spacing">3</property> - <child> - <object class="GtkCheckButton" id="pdf"> - <property name="label" translatable="yes" context="optprintpage|pdf">_PDF as standard print job format</property> - <property name="visible">True</property> - <property name="can-focus">True</property> - <property name="receives-default">False</property> - <property name="use-underline">True</property> - <property name="draw-indicator">True</property> - </object> - <packing> - <property name="left-attach">0</property> - <property name="top-attach">0</property> - </packing> - </child> <child> <object class="GtkCheckButton" id="converttogray"> <property name="label" translatable="yes" context="optprintpage|converttogray">Con_vert colors to grayscale</property> @@ -625,7 +611,7 @@ </object> <packing> <property name="left-attach">0</property> - <property name="top-attach">1</property> + <property name="top-attach">0</property> </packing> </child> <child> @@ -644,7 +630,7 @@ </object> <packing> <property name="left-attach">0</property> - <property name="top-attach">2</property> + <property name="top-attach">1</property> </packing> </child> </object> diff --git a/svtools/source/dialogs/restartdialog.cxx b/svtools/source/dialogs/restartdialog.cxx index 30954f17638b..e41ac165269d 100644 --- a/svtools/source/dialogs/restartdialog.cxx +++ b/svtools/source/dialogs/restartdialog.cxx @@ -32,9 +32,6 @@ public: case svtools::RESTART_REASON_JAVA: reason_ = m_xBuilder->weld_widget("reason_java"); break; - case svtools::RESTART_REASON_PDF_AS_STANDARD_JOB_FORMAT: - reason_ = m_xBuilder->weld_widget("reason_pdf"); - break; case svtools::RESTART_REASON_BIBLIOGRAPHY_INSTALL: reason_ = m_xBuilder->weld_widget("reason_bibliography_install"); break; diff --git a/svtools/uiconfig/ui/restartdialog.ui b/svtools/uiconfig/ui/restartdialog.ui index 9772c1659ad7..89fc4340533c 100644 --- a/svtools/uiconfig/ui/restartdialog.ui +++ b/svtools/uiconfig/ui/restartdialog.ui @@ -1,5 +1,5 @@ <?xml version="1.0" encoding="UTF-8"?> -<!-- Generated with glade 3.38.2 --> +<!-- Generated with glade 3.40.0 --> <interface domain="svt"> <requires lib="gtk+" version="3.20"/> <object class="GtkDialog" id="RestartDialog"> @@ -93,21 +93,6 @@ <property name="position">1</property> </packing> </child> - <child> - <object class="GtkLabel" id="reason_pdf"> - <property name="can-focus">False</property> - <property name="no-show-all">True</property> - <property name="label" translatable="yes" context="restartdialog|reason_pdf">For the modified default print job format to take effect, %PRODUCTNAME must be restarted.</property> - <property name="wrap">True</property> - <property name="max-width-chars">50</property> - <property name="xalign">0</property> - </object> - <packing> - <property name="expand">False</property> - <property name="fill">True</property> - <property name="position">2</property> - </packing> - </child> <child> <object class="GtkLabel" id="reason_bibliography_install"> <property name="can-focus">False</property>