[e2e] Update eslint guidelines to disallow page.locator in favor of web-first locators like page.getByRole

[unlikelyzero]

Summary

This will be a fairly large undertaking, but the playwright eslint plugin can help us find all examples of page.locator and replace with page.getByRole()

• [ ] Upgrade to eslint-playwright >1.6.0
• [ ] Set playwright eslint rule for no-nth-methods and no-raw-locators to required
• [ ] Skip for the performance/contract tests

[unlikelyzero]

Notable violations:

• Create Menu injects icon getByRole('menuitem', { name: ' Display Layout' })