@timhoffm, I've submitted the PR as per the approval. Kindly check and lmk if any changes are required.
Happy to update it. Thank you for the opportunity :)

Thanks@nrnavaneet, the typing change seems good but the new test seems unrelated.

Oh shoot ! Wrong test. I've made the changes.
Thanks for pointing that out !

If you are using AI to help with your contributions, please check everything it produces before submitting it. See alsohttps://matplotlib.org/devdocs/devel/contribute.html#restrictions-on-generative-ai-usage

I would not generally expect to see a test added with a typing change. In this case, I am in two minds whether to keep it. On one hand, test coverage is in general a good thing and this particular test is not expensive. On the other hand, the method in question is so simple that it is hard to see how it could go wrong, and there are existing tests that exercise it withintest_legend.py (although they are targeting other functionality).

Sorry !!, Still a new contributor. Im still figuring things out. Shldve checked before committing. My bad , wont happen again. It was simple idk why i made it so complex. If u want u can discard this pr :(.
Ive Read the rules, thanks again for pointing that out; it won't happen again. I love contributing to this repo.

Kindly lmk how u wanna proceed. I respect any decision you make

Ofcourse! Will do r8 away !

Removed the test as per the suggestion. Thank you :)