<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jan 14, 2014 at 10:40 PM, Juliusz Gonera <span dir="ltr"><<a href="mailto:jgonera@wikimedia.org" target="_blank">jgonera@wikimedia.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">In my opinion we should only have PageObjects for pages that actually exist</blockquote>

<div><br></div><div>I am fine with that, as long as the files do not become extremely big.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


There are a bunch of PageObjects that don't actually represent pages, but kind of mirror features:<br></blockquote><div><br></div><div>I actually prefer to have a lot of small classes instead of just a few big ones. If you prefer a few bigger classes, go ahead.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
I think all the elements from those page objects belong to the ArticlePage. Furthermore, to avoid repeating things that are shared everywhere (e.g. notifications icon), we could have a BasePage from which all the other page objects could inherit. What do you think?</blockquote>

<div><br></div>We already have some shared code in URL module[1]. I agree that moving shared code to a module makes sense.<br><br>Željko<br>--</div><div class="gmail_quote">1: <a href="https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/blob/master/tests/browser/features/support/modules/url_module.rb">https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/blob/master/tests/browser/features/support/modules/url_module.rb</a></div>

</div></div>