AllSpiceMirrors/kicad
7
Fork 0
mirror of https://gitlab.com/kicad/code/kicad.git synced 2025-04-05 00:15:30 +00:00
Files Issues Releases Activity

Use arrow not pencil when selecting image

Browse Source 
The arrow cursor makes better syntactic sense for placing existing items
rather than drawing as is the case for other tools

Fixes https://gitlab.com/kicad/code/kicad/-/issues/17588
This commit is contained in:
Seth Hillbrand 2024-09-13 16:51:16 -07:00
parent 89b9b13e21
commit c736904e39
3 changed files with 13 additions and 3 deletions
eeschema/tools
sch_drawing_tools.cpp
pagelayout_editor/tools
pl_drawing_tools.cpp
pcbnew/tools
drawing_tool.cpp

2
eeschema/tools/sch_drawing_tools.cpp
View File

@ -840,7 +840,7 @@ int SCH_DRAWING_TOOLS::PlaceImage( const TOOL_EVENT& aEvent )
if( image )
m_frame->GetCanvas()->SetCurrentCursor( KICURSOR::MOVING );
else
m_frame->GetCanvas()->SetCurrentCursor( KICURSOR::PENCIL );
m_frame->GetCanvas()->SetCurrentCursor( KICURSOR::ARROW );
};
auto cleanup =

12
pagelayout_editor/tools/pl_drawing_tools.cpp
View File

@ -85,8 +85,18 @@ int PL_DRAWING_TOOLS::PlaceItem( const TOOL_EVENT& aEvent )
{
if( item )
m_frame->GetCanvas()->SetCurrentCursor( KICURSOR::PLACE );
else if( isText )
{
m_frame->GetCanvas()->SetCurrentCursor( KICURSOR::TEXT );
}
else if( aEvent.IsAction( &PL_ACTIONS::placeImage ) )
{
m_frame->GetCanvas()->SetCurrentCursor( KICURSOR::ARROW );
}
else
m_frame->GetCanvas()->SetCurrentCursor( isText ? KICURSOR::TEXT : KICURSOR::PENCIL );
{
m_frame->GetCanvas()->SetCurrentCursor( KICURSOR::PENCIL );
}
};
auto cleanup =

2
pcbnew/tools/drawing_tool.cpp
View File

@ -578,7 +578,7 @@ int DRAWING_TOOL::PlaceReferenceImage( const TOOL_EVENT& aEvent )
if( image )
m_frame->GetCanvas()->SetCurrentCursor( KICURSOR::MOVING );
else
m_frame->GetCanvas()->SetCurrentCursor( KICURSOR::PENCIL );
m_frame->GetCanvas()->SetCurrentCursor( KICURSOR::ARROW );
};
auto cleanup =