<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 27, 2015 at 2:09 PM, Gehad Elrobey <span dir="ltr"><<a href="mailto:gehadelrobey@gmail.com" target="_blank">gehadelrobey@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Tue, May 26, 2015 at 12:55 PM, Lubomir I. Ivanov <span dir="ltr"><<a href="mailto:neolit123@gmail.com" target="_blank">neolit123@gmail.com</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"><span>On 25 May 2015 at 21:26, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com" target="_blank">gehadelrobey@gmail.com</a>> wrote:<br>
</span><span>> Hello Lubomir,<br>
><br>
> As GSoC coding period have begun, I wanted to update you with my progress<br>
> till now, I've pushed some more commits to my git branch, I will push some<br>
> more commits on the weekend as I am having my finals exams now. BTW Have you<br>
> managed to build subsurface with Grantlee yet or you still getting the same<br>
> linking error?<br>
><br>
<br>
</span>alright, i'm able to build now.<br>
<br>
please, merge the attached patch into your repository and let me know<br>
if you have any issues with it.<br>
also remove the NEW_PRINTING macro and CustomPrintDialog (more comments bellow).<br>
<br>
per-commit short reviews. read from bottom to top:<br>
<br>
---------------<br>
<br>
9b2701e Add Progress bar to custom print dialog<br>
<br>
see da5ce64<br>
<br>
3a37e57 Remove unused variable<br>
<br>
should be rebased. the code which it touches is WIP.<br>
<br>
ff97353 Open custom print dialog if NEW_PRINTING is defined<br>
<br>
see da5ce64<br>
<br>
da5ce64 Create the new custom print dialog<br>
<br>
i think you should just modify the current print dialog. adding more<br>
files creates noise.<br>
it was already discussed that master will have a "broken" printing<br>
module while this GSoC task is WIP.<br>
<br></blockquote></div></div><div><br>I though that I ll have to use the macro until the printing work is finished so that we can continue building subsurface without breaking the current module, I ll fix this.<br> </div><span class=""><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">
823d86a Add "NEW_PRINTING" Cmake variable<br>
<br>
like Dirk pointed out we are going to make grantlee a hard dependency<br>
for the printing to work. thus if cmake cannot find grantlee we just<br>
need to disable the printing.<br>
instead of the NEW_PRINTING cmake option rename it to NO_PRINTING (+<br>
change the description) which sets the NO_PRINTING pre-processor macro<br>
if "the user says so" or if grantlee is not found.<br>
<br>
ef99f65 Add Printer class that holds the rendering logic.<br>
<br>
this is WIP and is good to go as the basis for the future logic of<br>
having user printer dialog selectable properties.<br>
<br>
78fcd53 Add TemplateLayout class<br>
<br>
asked the same before; any comments on why we need a local Dive object?<br>
i highly doubt this will reach master as is.<br></blockquote></span><div><br>I needs the local dive object as Grantlee::Context is initialized by QVariantList<Dive> which contains all selected dives information, I have also checked the existing dive class at "<span style="font-size:12.8000001907349px">models.h" and the class was missing many fields that will be needed by the printing module, AFAIK the class in models.h is currently used in the dive list view only.</span></div></div></div></div></blockquote><div><br></div><div>What you need on the models? I can add them.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><span class="HOEnZb"><font color="#888888"><div><br></div>-- <br><div><div dir="ltr"><div><div dir="ltr">regards,</div></div><blockquote style="margin:0px 0px 0px 40px;border:none;padding:0px"><div><div dir="ltr"><div>Gehad</div></div></div></blockquote></div></div>
</font></span></div></div>
<br>_______________________________________________<br>
subsurface mailing list<br>
<a href="mailto:subsurface@subsurface-divelog.org">subsurface@subsurface-divelog.org</a><br>
<a href="http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface" target="_blank">http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface</a><br>
<br></blockquote></div><br></div></div>