Seems like you changed these around be accident?

(Same for the Enabled -> Active change below as well)

Ditto above ^

This name is a bit misleading as we don't allow duplicate types.
We create multiple edges from and edge that has multiple types assigned to it.

I think this it should be:

"If we allow duplicated edges, one edge should get added multiple times if is has been classified as more than one edge type. This is so we can create multiple different line type chains containing the same edge."

I think the description would be better if it was:

"Floating edges will be classified as contour lines"

Shouldn't this be in the chaining options though?
I don't think this should be in the "Geometry Processing" section as this only matters when chaining, right?

I think we should change the text string for "Image threshold" to something when this is on as we are not chaining in image space anymore. so it is on and "image space threshold" anymore.

I think this should be better in the "Edge type" section no?

From my point of view it feels better to have setting that states that and edge can have multiple types where the rest of the edge type selection. I'm not 100% though, so I'm up for discussing it quite a bit.

However this also doesn't seem to work. I made a file where the would be multiple overlapping types, but it doesn't seem to work.

I'll attach the file.

I'm guessing this need versioning code so we don't break older files?

Oh the values are already stored in that struct variable, it's just they happened to be defined in the wrong place. So no further action needed.

Although it happens in loading, but it makes sense to put into chaining panel. Yes

Should probably only be "Floating Edges" now right?
Otherwise you have "Chain: Chain floating edges" in the UI.

I'll use UI label in the draw call. Otherwise the property name itself is a bit confusing.