Not sure if we want to roll it into here or do it separately, but looks like the type hint for label_props should also be updated.

That type hint was based on reading the docstring, but it is clear from the code that Sequence/list was not actually a viable input to this parameter. (Namely, there is a check_is_instance for dict/None)

That was the one change that flagged to me as "oh, this is actually changing the documented type, we should be sure of that" and upon looking, I agree with the new version, just looking to keep the type hints in sync if we can.

Thanks for noticing this @ksunden. I think it is appropriate to fix this in this PR too - type hints are sort of a documentation. Pushed a fix.