haskal
/
kicad
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

DIALOG_PAD_PROPERTIES: fix incorrect pad type for Aperture option. 

The aperture option created a Connector pad only on the paste layer.
This is incorrect: the SMD type only is acceptable.
5connector is *only* for edge card connectors.
Moreover, The dialog did not even accept the setting defined by this dialog...
This commit is contained in:
jean-pierre charras 2019-11-17 18:20:19 +01:00
parent a4c9f961f7
commit eadcb5de5e
3 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
pcbnew/dialogs/dialog_pad_properties.cpp
View File

@ -79,7 +79,8 @@ static PAD_ATTR_T code_type[] =
PAD_ATTRIB_SMD,
PAD_ATTRIB_CONN,
PAD_ATTRIB_HOLE_NOT_PLATED,
PAD_ATTRIB_CONN // Aperture pad (type CONN with no copper layers)
PAD_ATTRIB_SMD // Aperture pad :type SMD with no copper layers,
// only on tech layers (usually only on paste layer
};
// Default mask layers setup for pads according to the pad type

2
pcbnew/dialogs/dialog_pad_properties_base.cpp
View File

@ -62,7 +62,7 @@ DIALOG_PAD_PROPERTIES_BASE::DIALOG_PAD_PROPERTIES_BASE( wxWindow* parent, wxWind
m_staticText44->Wrap( -1 );
fgSizerShapeType->Add( m_staticText44, 0, wxALIGN_CENTER_VERTICAL|wxTOP|wxBOTTOM|wxLEFT, 3 );
wxString m_PadTypeChoices[] = { _("Through-hole"), _("SMD"), _("Edge Connector"), _("NPTH, Mechanical"), _("Aperture") };
wxString m_PadTypeChoices[] = { _("Through-hole"), _("SMD"), _("Edge Connector"), _("NPTH, Mechanical"), _("SMD Aperture") };
int m_PadTypeNChoices = sizeof( m_PadTypeChoices ) / sizeof( wxString );
m_PadType = new wxChoice( m_panelGeneral, wxID_ANY, wxDefaultPosition, wxDefaultSize, m_PadTypeNChoices, m_PadTypeChoices, 0 );
m_PadType->SetSelection( 0 );

2
pcbnew/dialogs/dialog_pad_properties_base.fbp
View File

@ -561,7 +561,7 @@
<property name="caption"></property>
<property name="caption_visible">1</property>
<property name="center_pane">0</property>
<property name="choices">&quot;Through-hole&quot; &quot;SMD&quot; &quot;Edge Connector&quot; &quot;NPTH, Mechanical&quot; &quot;Aperture&quot;</property>
<property name="choices">&quot;Through-hole&quot; &quot;SMD&quot; &quot;Edge Connector&quot; &quot;NPTH, Mechanical&quot; &quot;SMD Aperture&quot;</property>
<property name="close_button">1</property>
<property name="context_help"></property>
<property name="context_menu">1</property>