<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jan 21, 2018 at 1:30 PM, Jan Mulder <span dir="ltr"><<a href="mailto:jlmulder@xs4all.nl" target="_blank">jlmulder@xs4all.nl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 21-01-18 12:50, Tomaz Canabrava wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This is something that I didn't wanted to do since the beginning.<br>
our models *should* be both desktop and mobile compatible (with the exception of the dive list because of the way QT handles Tree Views)<br>
</blockquote>
<br></span>
I agree with that (models *should* be both desktop and mobile compatible). However, when checking the current code base, we see that we only have 2 models that are used both in mobile and desktop, and the rest is used exclusively only on mobile (3 models) or only in desktop (18! models). And as there is no reason to link in any unused code, just do not link those 18 desktop only models to the mobile app.<span class=""><br></span></blockquote><div><br></div><div>there's also no reason to duplicate code, I prefer to have a bit of unused code in both platforms than to duplicate stuff - also this means less code to maintain, and usually less bugs.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
if there's a mobile specific model, we should try to look if it's not a mistake and if a corresponding model already exists doing the exact same thing, and only have one model for each thing.<br>
</blockquote>
<br></span>
Also true, but we could ask the same question for "why desktop specific model". In numerous cases it is valid to have desktop only models, as there is much more functionality in the desktop than in the mobile app.<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>that code increase is not so big to matter, I think<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
<br>
--jan<br>
</font></span></blockquote></div><br></div></div>