components duplicated

[frankrolf]

When using overlapper in a glyph with both contours and components, components are added on commit – on top of existing components. This means that a glyph may end up with multiple overlapping components (befitting the name of the extension, but probably unintentional).

I solved this by just commenting out https://github.com/ryanbugden/Overlapper/blob/main/Overlapper.roboFontExt/lib/overlapper.py#L563-L565

What is the purpose of the stored_components workflow? You could either clear components before re-adding them, or just leave out that logic. I see some component work in #2, but it seems that wasn’t considered important.

[ryanbugden]

Thanks for this, Frank! Haha I'll have to make a fork called Component Overlapper. I think this was a holdover from when I was operating on the whole glyph rather than just the selected contours. I've commented it out for now and kept it messy, as a reminder to go back in and clean everything up better later when I have more time. Leaving this open.