<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">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 class="">On 25 May 2015 at 21:26, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
</span><span class="">> 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><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><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><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><br></div>-- <br><div class="gmail_signature"><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>
</div></div>