Hi Stuart, I’m not an official captial-D KiCad Developer, but I have submitted a fix for a different issue. It was as simple as:
1. Log an issue in gitlab. Follow the instructions in the issue template, and especially make sure the steps to reproduce are clear. Include images or gifs if helpful. Make sure you explain why the reported behaviour is a problem. 2. Fork the repo on gitlab. 3. Clone your forked repo and apply the fix in a branch. 4. Push your branch to your fork. 5. Open a pull request. Follow the instructions in the template, and be sure the before and after behaviour is clear. Again, images or gifs may be helpful. 6. Wait for comments. 7. Address any comments. 8. When the fix has been approved, an official developer will merge it. And of course, you should exhaustively test your fix. Unfortunately KiCad doesn’t have automated tests for UI, so the onus is on you to ensure your patch fixes the problem AND doesn’t introduce any regressions. If you haven’t already, make sure you read the KiCad developer culture page: https://dev-docs.kicad.org/en/rules-guidelines/culture/ > On Jul 6, 2023, at 11:07 AM, Stuart Tyler <[email protected]> wrote: > > I have been digging and understanding more - I believe line 717 in > edit_tool_move_fct.cpp should be: > > nextItem->Move( controls->GetMousePosition( true ) - > nextItem->GetPosition()); > > This moves the item to the current cursor, without destroying the shape of > the track if the item is a track. > A set position method would be really nice to have, but this in effect does > the same thing. > > Feel free to direct me on what to do with this information. I am just getting > my feet and Im an adult, so just let me know directly if posting this > information here is the wrong thing to do. I think I found an issue, and the > solution, but what to do next - I have no idea. > > Regards > Stuart > > > On Thursday, July 6, 2023 at 3:27:09 PM UTC+1 Stuart Tyler wrote: >> Hi, >> I have my KiCad Windows development system going and browsing my way through >> pcbnew, getting up to speed on the internal workings of moving and dragging. >> >> I think I have observed an issue in moveIndividually, the ctrl-M command. >> When you select multiple traces, the first trace is moved nicely, but every >> trace after that is mangled. I believe this is because in >> edit_tool_move_fct.cpp on line 717: >> nextItem->SetPosition( controls->GetMousePosition( true ) ); >> >> When the item is a track, only one end of the track is moved, and so now the >> tack shape is mangled. >> >> When this line is commented out, the track shape is maintained, but the grab >> position is not at the track and is really annoying. As this is my first >> playing with KiCad and pcbnew, I am unsure on how to make the mouse position >> warp to the current track location, or move the track to the current mouse >> location without damaging the track shape. >> >> This would then fix the moveIndividually ctrl-M command for tracks. >> >> Regards >> Stuart > > > -- > You received this message because you are subscribed to the Google Groups > "KiCad Developers" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected] <mailto:[email protected]>. > To view this discussion on the web visit > https://groups.google.com/a/kicad.org/d/msgid/devlist/f51b36b1-e094-4727-b0cf-5e6f56c678a1n%40kicad.org > > <https://groups.google.com/a/kicad.org/d/msgid/devlist/f51b36b1-e094-4727-b0cf-5e6f56c678a1n%40kicad.org?utm_medium=email&utm_source=footer>. -- You received this message because you are subscribed to the Google Groups "KiCad Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/a/kicad.org/d/msgid/devlist/647D8CD4-69F3-4ABE-85B2-D4B5154EE6BE%40gmail.com.
