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

router: improve heuristic to avoid segment drag 'flickery snapping' 

Fixes https://gitlab.com/kicad/code/kicad/-/issues/10113
This commit is contained in:
Tomasz Wlostowski 2022-12-29 12:16:25 +01:00
parent 157b93c4ae
commit dfb1068358
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
pcbnew/router/pns_tool_base.cpp
View File

@ -264,12 +264,13 @@ bool TOOL_BASE::checkSnap( ITEM *aItem )
// Sync PNS engine settings with the general PCB editor options. // Sync PNS engine settings with the general PCB editor options.
auto& pnss = m_router->Settings(); auto& pnss = m_router->Settings();
// If we're dragging a track segment, don't try to snap to items on the same copper layer. // If we're dragging a track segment, don't try to snap to items on the same copper layer with same nets. This is not a perfect heuristic, but seems to work reasonably well :-)
// This way we avoid 'flickery' behaviour for short segments when the snap algo is trying to // This way we avoid 'flickery' behaviour for short segments when the snap algo is trying to
// snap to the corners of the segments next to the one being dragged. // snap to the corners of the segments next to the one being dragged.
if( m_startItem && aItem && m_router->GetState() == ROUTER::DRAG_SEGMENT if( m_startItem && aItem && m_router->GetState() == ROUTER::DRAG_SEGMENT
&& aItem->Layer() == m_startItem->Layer() && aItem->OfKind( ITEM::SEGMENT_T ) && aItem->Layer() == m_startItem->Layer() && aItem->OfKind( ITEM::SEGMENT_T )
&& m_startItem->OfKind( ITEM::SEGMENT_T ) ) && m_startItem->OfKind( ITEM::SEGMENT_T )
&& aItem->Net() == m_startItem->Net() )
return false; return false;
pnss.SetSnapToPads( pnss.SetSnapToPads(