diff --git a/common/jobs/job_export_pcb_dxf.cpp b/common/jobs/job_export_pcb_dxf.cpp index b63e80077f..b4acefbc82 100644 --- a/common/jobs/job_export_pcb_dxf.cpp +++ b/common/jobs/job_export_pcb_dxf.cpp @@ -33,22 +33,11 @@ JOB_EXPORT_PCB_DXF::JOB_EXPORT_PCB_DXF() : m_plotGraphicItemsUsingContours( true ), m_dxfUnits( DXF_UNITS::INCHES ) { - m_plotDrawingSheet = false; - - m_params.emplace_back( - new JOB_PARAM<wxString>( "drawing_sheet", &m_drawingSheet, m_drawingSheet ) ); m_params.emplace_back( new JOB_PARAM<bool>( "plot_footprint_values", &m_plotFootprintValues, m_plotFootprintValues ) ); - m_params.emplace_back( new JOB_PARAM<bool>( "plot_ref_des", &m_plotRefDes, m_plotRefDes ) ); m_params.emplace_back( new JOB_PARAM<bool>( "plot_graphic_items_using_contours", &m_plotGraphicItemsUsingContours, m_plotGraphicItemsUsingContours ) ); - m_params.emplace_back( - new JOB_PARAM<bool>( "use_drill_origin", &m_useDrillOrigin, m_useDrillOrigin ) ); - m_params.emplace_back( new JOB_PARAM<bool>( "plot_border_title_blocks", &m_plotDrawingSheet, - m_plotDrawingSheet ) ); m_params.emplace_back( new JOB_PARAM<DXF_UNITS>( "units", &m_dxfUnits, m_dxfUnits ) ); - m_params.emplace_back( - new JOB_PARAM<LSEQ>( "layers", &m_printMaskLayer, m_printMaskLayer ) ); } diff --git a/common/jobs/job_export_pcb_gerber.cpp b/common/jobs/job_export_pcb_gerber.cpp index 8f011aaf18..f29dcd1a49 100644 --- a/common/jobs/job_export_pcb_gerber.cpp +++ b/common/jobs/job_export_pcb_gerber.cpp @@ -41,12 +41,6 @@ JOB_EXPORT_PCB_GERBER::JOB_EXPORT_PCB_GERBER( const std::string& aType ) : &m_plotFootprintValues, m_plotFootprintValues ) ); - m_params.emplace_back( new JOB_PARAM<bool>( "plot_ref_des", &m_plotRefDes, m_plotRefDes ) ); - - m_params.emplace_back( new JOB_PARAM<bool>( "plot_drawing_sheet", - &m_plotDrawingSheet, - m_plotDrawingSheet ) ); - m_params.emplace_back( new JOB_PARAM<bool>( "subtract_solder_mask_from_silk", &m_subtractSolderMaskFromSilk, @@ -59,14 +53,10 @@ JOB_EXPORT_PCB_GERBER::JOB_EXPORT_PCB_GERBER( const std::string& aType ) : m_params.emplace_back( new JOB_PARAM<bool>( "use_x2_format", &m_useX2Format, m_useX2Format ) ); m_params.emplace_back( new JOB_PARAM<bool>( "disable_aperture_macros", &m_disableApertureMacros, m_disableApertureMacros ) ); - m_params.emplace_back( new JOB_PARAM<bool>( "use_aux_origin", - &m_useDrillOrigin, - m_useDrillOrigin ) ); m_params.emplace_back( new JOB_PARAM<bool>( "use_protel_file_extension", &m_useProtelFileExtension, m_useProtelFileExtension ) ); m_params.emplace_back( new JOB_PARAM<int>( "precision", &m_precision, m_precision ) ); - m_params.emplace_back( new JOB_PARAM<LSEQ>( "layers", &m_printMaskLayer, m_printMaskLayer ) ); } diff --git a/common/jobs/job_export_pcb_gerbers.cpp b/common/jobs/job_export_pcb_gerbers.cpp index 5e2ca32b94..4646e8033c 100644 --- a/common/jobs/job_export_pcb_gerbers.cpp +++ b/common/jobs/job_export_pcb_gerbers.cpp @@ -30,9 +30,6 @@ JOB_EXPORT_PCB_GERBERS::JOB_EXPORT_PCB_GERBERS() : m_useBoardPlotParams( false ), m_createJobsFile( true ) { - m_params.emplace_back( new JOB_PARAM<bool>( "use_board_plot_params", &m_useBoardPlotParams, - m_useBoardPlotParams ) ); - m_params.emplace_back( new JOB_PARAM<bool>( "layers_include_on_all_set", &m_layersIncludeOnAllSet, m_layersIncludeOnAllSet ) ); @@ -43,7 +40,7 @@ JOB_EXPORT_PCB_GERBERS::JOB_EXPORT_PCB_GERBERS() : wxString JOB_EXPORT_PCB_GERBERS::GetDefaultDescription() const { - return wxString::Format( _( "Multi gerber export" ) ); + return wxString::Format( _( "PCB Gerbers export" ) ); } diff --git a/common/jobs/job_export_pcb_pdf.cpp b/common/jobs/job_export_pcb_pdf.cpp index 11d83e0a8a..16dd93244b 100644 --- a/common/jobs/job_export_pcb_pdf.cpp +++ b/common/jobs/job_export_pcb_pdf.cpp @@ -32,27 +32,20 @@ NLOHMANN_JSON_SERIALIZE_ENUM( JOB_EXPORT_PCB_PDF::DRILL_MARKS, JOB_EXPORT_PCB_PDF::JOB_EXPORT_PCB_PDF() : JOB_EXPORT_PCB_PLOT( JOB_EXPORT_PCB_PLOT::PLOT_FORMAT::PDF, "pdf", false ) { - m_plotDrawingSheet = false; - m_params.emplace_back( new JOB_PARAM<wxString>( "color_theme", &m_colorTheme, m_colorTheme ) ); - m_params.emplace_back( - new JOB_PARAM<wxString>( "drawing_sheet", &m_drawingSheet, m_drawingSheet ) ); m_params.emplace_back( new JOB_PARAM<bool>( "mirror", &m_mirror, m_mirror ) ); m_params.emplace_back( new JOB_PARAM<bool>( "black_and_white", &m_blackAndWhite, m_blackAndWhite ) ); m_params.emplace_back( new JOB_PARAM<bool>( "negative", &m_negative, m_negative ) ); m_params.emplace_back( new JOB_PARAM<bool>( "plot_footprint_values", &m_plotFootprintValues, m_plotFootprintValues ) ); - m_params.emplace_back( new JOB_PARAM<bool>( "plot_ref_des", &m_plotRefDes, m_plotRefDes ) ); - m_params.emplace_back( new JOB_PARAM<bool>( "plot_border_title_blocks", &m_plotDrawingSheet, - m_plotDrawingSheet ) ); - m_params.emplace_back( new JOB_PARAM<LSEQ>( "layers", &m_printMaskLayer, m_printMaskLayer ) ); + m_params.emplace_back( new JOB_PARAM<bool>( "plot_pad_numbers", &m_plotPadNumbers, m_plotPadNumbers ) ); + m_params.emplace_back( new JOB_PARAM<bool>( "plot_invisible_text", &m_plotInvisibleText, + m_plotInvisibleText ) ); m_params.emplace_back( new JOB_PARAM<bool>( "sketch_pads_on_fab_layers", &m_sketchPadsOnFabLayers, m_sketchPadsOnFabLayers ) ); m_params.emplace_back( new JOB_PARAM<DRILL_MARKS>( "drill_shape", &m_drillShapeOption, m_drillShapeOption ) ); - m_params.emplace_back( - new JOB_PARAM<bool>( "use_drill_origin", &m_useDrillOrigin, m_useDrillOrigin ) ); } diff --git a/common/jobs/job_export_pcb_pdf.h b/common/jobs/job_export_pcb_pdf.h index 34edea6a5d..e8cc5efd4d 100644 --- a/common/jobs/job_export_pcb_pdf.h +++ b/common/jobs/job_export_pcb_pdf.h @@ -33,8 +33,6 @@ class KICOMMON_API JOB_EXPORT_PCB_PDF : public JOB_EXPORT_PCB_PLOT public: JOB_EXPORT_PCB_PDF(); wxString GetDefaultDescription() const override; - - bool m_useDrillOrigin; }; #endif diff --git a/common/jobs/job_export_pcb_plot.cpp b/common/jobs/job_export_pcb_plot.cpp index 39bb22ea24..e6aee90fcd 100644 --- a/common/jobs/job_export_pcb_plot.cpp +++ b/common/jobs/job_export_pcb_plot.cpp @@ -26,5 +26,25 @@ JOB_EXPORT_PCB_PLOT::JOB_EXPORT_PCB_PLOT( PLOT_FORMAT aFormat, const std::string m_drillShapeOption( DRILL_MARKS::FULL_DRILL_SHAPE ), m_useDrillOrigin( false ) { + m_params.emplace_back( new JOB_PARAM<LSEQ>( "layers", &m_printMaskLayer, m_printMaskLayer ) ); + m_params.emplace_back( new JOB_PARAM<LSEQ>( "layers_to_include_on_all_layers", + &m_printMaskLayersToIncludeOnAllLayers, + m_printMaskLayersToIncludeOnAllLayers ) ); + + m_params.emplace_back( + new JOB_PARAM<bool>( "plot_pad_numbers", &m_plotPadNumbers, m_plotPadNumbers ) ); + m_params.emplace_back( new JOB_PARAM<bool>( "plot_invisible_text", &m_plotInvisibleText, + m_plotInvisibleText ) ); + + m_params.emplace_back( + new JOB_PARAM<bool>( "plot_drawing_sheet", &m_plotDrawingSheet, m_plotDrawingSheet ) ); + + m_params.emplace_back( new JOB_PARAM<bool>( "plot_ref_des", &m_plotRefDes, m_plotRefDes ) ); + + m_params.emplace_back( + new JOB_PARAM<bool>( "use_drill_origin", &m_useDrillOrigin, m_useDrillOrigin ) ); + + m_params.emplace_back( + new JOB_PARAM<wxString>( "drawing_sheet", &m_drawingSheet, m_drawingSheet ) ); } \ No newline at end of file diff --git a/common/jobs/job_export_pcb_svg.cpp b/common/jobs/job_export_pcb_svg.cpp index fe30e17e07..a864f3da8d 100644 --- a/common/jobs/job_export_pcb_svg.cpp +++ b/common/jobs/job_export_pcb_svg.cpp @@ -31,15 +31,10 @@ JOB_EXPORT_PCB_SVG::JOB_EXPORT_PCB_SVG() : m_plotDrawingSheet = true; m_params.emplace_back( new JOB_PARAM<wxString>( "color_theme", &m_colorTheme, m_colorTheme ) ); - m_params.emplace_back( - new JOB_PARAM<wxString>( "drawing_sheet", &m_drawingSheet, m_drawingSheet ) ); m_params.emplace_back( new JOB_PARAM<bool>( "mirror", &m_mirror, m_mirror ) ); m_params.emplace_back( new JOB_PARAM<bool>( "black_and_white", &m_blackAndWhite, m_blackAndWhite ) ); m_params.emplace_back( new JOB_PARAM<bool>( "negative", &m_negative, m_negative ) ); - m_params.emplace_back( - new JOB_PARAM<bool>( "plot_drawing_sheet", &m_plotDrawingSheet, m_plotDrawingSheet ) ); - m_params.emplace_back( new JOB_PARAM<LSEQ>( "layers", &m_printMaskLayer, m_printMaskLayer ) ); m_params.emplace_back( new JOB_PARAM<bool>( "sketch_pads_on_fab_layers", &m_sketchPadsOnFabLayers, m_sketchPadsOnFabLayers ) ); m_params.emplace_back( @@ -47,8 +42,6 @@ JOB_EXPORT_PCB_SVG::JOB_EXPORT_PCB_SVG() : m_params.emplace_back( new JOB_PARAM<DRILL_MARKS>( "drill_shape", &m_drillShapeOption, m_drillShapeOption ) ); m_params.emplace_back( new JOB_PARAM<unsigned int>( "precision", &m_precision, m_precision ) ); - m_params.emplace_back( - new JOB_PARAM<bool>( "use_drill_origin", &m_useDrillOrigin, m_useDrillOrigin ) ); } diff --git a/pcbnew/dialogs/dialog_plot.cpp b/pcbnew/dialogs/dialog_plot.cpp index 073d15bc73..9d3253dd8d 100644 --- a/pcbnew/dialogs/dialog_plot.cpp +++ b/pcbnew/dialogs/dialog_plot.cpp @@ -1273,7 +1273,7 @@ void DIALOG_PLOT::Plot( wxCommandEvent& event ) { applyPlotSettings(); transferPlotParamsToJob(); - Close(); + EndModal( wxID_OK ); } else {