Add new "no element handle" rule

[elaichenkov]

Adds new no-element-handle rule.

Description:
This rule disallows using the page.$ element handle and page.$$ element handles. Because the use of ElementHandle is discouraged, use Locator instead. Besides, it can convert an element handle to a locator.

[elaichenkov]

@mskelton when you get a chance please take a look at PR once again. I've updated it.

[mxschmitt]

Maybe we should also discourage from $eval and $$eval?

[elaichenkov]

Maybe we should also discourage from $eval and $$eval?

Yeah, sure. However, I'd create a new no-eval-method or no-eval rule for it, which would prevent usage of page.$eval and page.$$eval. How do you feel about this?

[mskelton]

Great work!

[elaichenkov]

Great work!

Thank you! Btw, I'd love to start working on the "page.$eval and page.$$eval" rule. What do you think is a better name for this?

1. no-eval-method
2. no-eval-function
3. no-eval

[mskelton]

no-eval seems fine with me. It's scoped to this package, so it would be playwright/no-eval so there isn't an issue with it conflicting with the built-in no-eval rule.

[mskelton]

Alternatively, it could be no-page-eval similar to no-page-pause 🤷‍♂️