This codifies current behavior. It does not fix#29690, but is a baseline so that we can do a reliable refactoring to fix#29690 without accidentally changing current behavior (see#29261 (comment)).

@timhoffm In the current behavior, whenpatch.force_edgecolor=False andfacecolor="none", the artist becomes fully transparent (invisible). Should we first address this specific issue in the existingpatch.force_edgecolor logic, or should we focus on#29261?

Sorry, I always get a knot in my brain when I try to think though all cases 😵. The fundamental goal of#29261 is to get rid of the confusingpatch.force_edgecolor. I'm not clear whether we need to to fix a behavior that depends on that first. Ultimately, we want new parameters

#patch.edgecolor: "none"            # default edgecolor for Patch and Collection#patch.edgecolor_fallback: "black"  # edgecolor to be used if facecolor is "none" to prevent completely invisible artists

and an as-smooth-as possible transition path to them - i.e. give themes/users with customizedpatch.force_edgecolor/patch_edgecolor alternatives to configure their behavior through the new parameters.

I suspect, but right now can't tell reliably, that changing part of the logic ofpatch.force_edgecolor before does not add a benefit because that'll be a breaking change, and we can as well do the break during the migration if needed. Therefore, I suggest that you move forward on#29261 and we'll see how well that goes. At worst, we'll have to take a step back, do the changepatch.force_edgecolor and then come back to#29261 afterwards.