PNS tune status popup is not drawn initially

Bug #1428285 reported by Nick Østergaard
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Fix Released
Low
Tomasz Wlostowski

Bug Description

The push and shove routers tune status popup is not displayed directly after mouse click.

An related issue with it drawing in the screencoordinates origin was fixed in 5474 and refined in 5475. I don't think the behaivour here is user friendly.

What happens is that it will not show until the mouse curosr is moved, and this is problematic, because this can cause

I think the correct status popup should appear even if the cursor is not moved. It could make people think there is a sporadic issue with the dialog at times, if they wonder why it does not show up instantly. This because mostly people might move the mouse a little after click, but not always.

Steps to reproduce:
1. Make a trace in GAL
2. Use the option "Route -> Tune track length"
3. Click the trace drawn previosly (or any trace)
3a. It is important not to move the mouse cursor to see that the popup is not there (together with the yellow highlighting line thingie)
3b. Move the cursor to see the stuff appear

Unexpected result:
See step 3a and 3b.

Expected result:
The wish is to make step 3 into just showing the popup and the other highlight stuff immediately after click and not just on cursor move.

This was tested on 5475.

Tags: cern pcbnew pns

Related branches

Changed in kicad:
assignee: nobody → Tomasz Wlostowski (twlostow)
Changed in kicad:
status: New → Fix Committed
Changed in kicad:
importance: Undecided → Low
Revision history for this message
Nick Østergaard (nickoe) wrote :

Yay, I confirm this to be fixed, looks good to me now. Thanks.

Jon Neal (reportingsjr)
Changed in kicad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.