changed footprint name isn't reflected in the tree

[eelik]

If I change the footprint name in Footprint Library Editor's "Footprint properties" nothing seems to happen. Actually the footprint exists with the new name, but it's visible in the tree view only after closing and reopening the editor, even if it is saved after renaming. The most logical behavior would be to actually change the name without copying the footprint. Copying with a new name would be OK (and safe) but it should be visible in the tree view immediately and it should then be the active footprint.

[KiCad Janitor (kicad-janitor)]

Fixed in revision e5e1a315f19a612bc0719a5c27f2750edcb68c43
https://git.launchpad.net/kicad/patch/?id=e5e1a315f19a612bc0719a5c27f2750edcb68c43

[eelik (eelik)]

This seems to work logically now. The changed name is saved without confirmation, even if the editor is closed without saving, so I hope it doesn't bite anyone. But it does what is expected.

[Jeff Young (jeyjey)]

Hmmm... that's not great. I think I need to add a confirmation dialog.

[Jeff Young (jeyjey)]

Well, the confirmation dialog was clunky. It just felt wrong.

I've managed to finesse it such that it doesn't save the changes in the module (and library tree) until you save.

[KiCad Janitor (kicad-janitor)]

Fixed in revision a5e2ddd65a38f9677bc1667a68d9bd06302678ce
https://git.launchpad.net/kicad/patch/?id=a5e2ddd65a38f9677bc1667a68d9bd06302678ce

[Jeff Young (jeyjey)]

@eelik, if you could give this new version a spin I'd appreciate it.

Cheers,
Jeff.

[eelik (eelik)]

Now it feels right and natural, all changes including renaming are equal. I tested different scenarios of opening, modifying, renaming and saving footprints, and can't find a way to make it fail. Together with fixes for #1792256 everything works fine.