<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jun 20, 2015 at 11:03 AM, 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 20 June 2015 at 06:19, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
><br>
><br>
> On Fri, Jun 19, 2015 at 9:05 PM, Lubomir I. Ivanov <<a href="mailto:neolit123@gmail.com">neolit123@gmail.com</a>><br>
> wrote:<br>
>><br>
>> So this is the second batch of patches for this GSoC task.<br>
>><br>
>> Here Gehad has added another template which is the "one dive per page",<br>
>> some<br>
>> minor fixes and renames here and there and also support for printing on<br>
>> any<br>
>> page size (or "render target" size).<br>
>><br>
>> Also good news is that I'm now actually able to use Grantlee properly<br>
>> on Windows and the printouts are rendered. Also the NO_PRINTING option<br>
>> actually<br>
>> works this time.<br>
>><br>
>> Gehad, a couple of things:<br>
>><br>
>> - Again, I did some capitalization fixes and fixed small typos in the<br>
>> commit messages. Nothing big.<br>
>><br>
>> - Patch headings such as "Printing: " should then preferably be followed<br>
>> with a lower case word e.g. "Printing: something in this patch" instead of<br>
>> "Printing: Something in this patch"<br>
>><br>
><br>
> I'll take care of this in my coming commit messages.<br>
><br>
<br>
</span>thanks<br>
<span class=""><br>
>><br>
>> - It would be great if you can implement the Print->Preview dialog that we<br>
>> had, since that was very useful not to generate a PDF to be able to view<br>
>> how it *actually* renders.<br>
>><br>
><br>
> I am planing to implement the preview dialog in the "TemplateEdit" class so<br>
> I think this will be finished in the next two weeks.<br>
><br>
<br>
</span>the QPrintPreview dialog which is triggered from the main print dialog<br>
is another thing...<br>
this renders all (selected) dives on pages on a render target which is<br>
show in the dialog, from which the user can modify the page layout and<br>
actually print! this is an actual print preview and we need the button<br>
to remain where it is ATM. go back to my previous implementation and<br>
see how easy it was to do it. i may have missed why you removed the<br>
QPrintPreview trigger from printdialog.cpp in the first place. :-)<br>
<br>
having a preview in TemplateEdit is another thing, which should be a<br>
QPixmap preview of said template for one, two or more dives just as an<br>
image placed in said dialog, or as a larger preview window - QWebView<br>
if you will.<br>
<span class=""><br>
>><br>
>> - Supporting the "print selected dives" properly is easy to implement and<br>
>> nice to have while testing.<br>
>><br>
><br>
> I am going to work on this in the current week.<br>
><br>
<br>
</span>cool<br>
<span class=""><br>
>><br>
>> - Not sure we have discussed a "Statistics print" for GSoC. Is this<br>
>> something<br>
>> which came from user requests - e.g. from IRC?<br>
>><br>
><br>
> That was mentioned in the proposal, we can cancel this feature if you would<br>
> suggest this.<br>
><br>
<br>
</span>will comment more on Dirk's comment for this, but consider it a<br>
nice-to-have feature and should be moved towards the end of the<br>
schedule.<br>
<span class=""><br>
>><br>
>> - Can you update us with your TODO list or at least with the next few<br>
>> steps out<br>
>> of it?<br>
><br>
><br>
> here is my list:<br>
> - Fixing the dive profile rendering bugs (mentioned in a previous email).<br>
<br>
</span>it might be a good idea to get one of the templates soon to look<br>
user-ready - e.g. the "two dives" one.<br>
<span class=""><br>
> - Implementing the flow layout template.<br>
<br>
</span>i remember this one; should be less of a priority for now.<br>
my idea is the following, get *everything* core wise working and one<br>
of the templates user-ready to showcase your work and only then, start<br>
adding more templates and options.<br>
<span class=""><br>
> - Implement the remaining features in the "PrintOptions" class (Including<br>
> "print selected dives" option).<br>
<br>
</span>please move this towards the top.<br>
<span class=""><br>
> - Implement the "TemplateEdit" Class (Including the print preview).<br>
><br>
<br>
</span>should be higher priority than adding more templates.<br>
also, please add the QPrintPreviewDialog implementation on top of the<br>
list, as it makes it *much* easier to test the whole project.<br>
<br>
here is how i see it:<br>
- QPrintPreviewDialog triggered from the main dialog<br>
- "print selected dives" support<br>
- TemplateEdit with print preview<br>
- get one of the templates user-ready (two dives)<br>
- more print options - e.g. table print<br>
- more templates - e.g. 'flow"<br>
- work on mailing list feedback from users<br>
- statistics print<br></blockquote><div><br>okay, I'll stuck to this priority list.</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>