Remove inheritance from e2e page models

[cquinn540]

Terms

• I have searched open and closed design issues
• I agree to follow activist's Code of Conduct

Description

Currently the page objects models use a combination of inheritance and pseudo-inheritance via a Proxy that make the code difficult to debug and maintain.

The page objects should be simple and just use composition with a page field and fields for components on the page that are their own objects. Also the use of key value pairs for the locators makes the tests harder to read.

Contribution

I will work on this.