<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Aug 15, 2015 at 1:59 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 14 August 2015 at 23:54, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
><br>
><br>
> On Thu, Aug 13, 2015 at 10:47 AM, Lubomir I. Ivanov <<a href="mailto:neolit123@gmail.com">neolit123@gmail.com</a>><br>
> wrote:<br>
>><br>
>> On 12 August 2015 at 18:47, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
>> > Hello Lubomir,<br>
>> ><br>
>> > I have resolved most of the issues you have mentioned above, still<br>
>> > working<br>
>> > on enhancing the statistics view, I hope I didn't miss anything so<br>
>> > please<br>
>> > have a look and update me if you have any notes.<br>
>> > All commits are pushed to my branch.<br>
>> ><br>
>><br>
>> most of the templates look good, but there are still some issues.<br>
>> also, i think we are missing the logic to tell the viewport where a<br>
>> new HTML page starts. would JS be needed for that? let me know if you<br>
>> need ideas about this one [1].<br>
>><br>
>> let's enumerate the issues for better tracking:<br>
>><br>
>> 1. all templates: the page calculation still leaves a couple of pixels<br>
>> at the end of the page blank. i think you may have a slight rounding<br>
>> error in the viewport code.<br>
>> see 0.jpg<br>
<br>
</span>re: "Printing: remove silly white line at end of each page"<br>
<br>
so this is a Qt < 5.3 issue and on such older versions the page page<br>
calculation will be slightly off and the white pixels will appear - am<br>
i right?<br>
if so, can we adjust the calculation somehow for Qt < 5.3?<br>
<span class=""><br></span></blockquote><div><br>I could fix the issue with <font color="#000000">QPrinter::</font><span style="color:rgb(0,0,0)">pageRect which happened because of float to int casting, I will replace pageLayout for now and calculate the resolution with only pageRect for all Qt versions.</span></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>
>> 2. table: the table is still cut on a new page. can we make it so that<br>
>> the new page always starts at a new row [1]?<br>
>> see 1.jpg<br>
>><br>
>> 3. flow: cut on a new page [1]<br>
>> see 2.jpg<br>
>><br>
>> 4. flow: use the same background for all the data cells<br>
>> see 3.jpg<br>
>><br>
>> 5. one dive: use the same background for all the data cells<br>
>> (similar to 4.)<br>
>><br>
>> 6. six dives: the table is missing backgrounds and borders. please<br>
>> style it the same as the tables in the other templates.<br>
>> see 4.jpg<br>
>><br>
>> 7. two dives: add more text padding on the left (margins)<br>
>> see 5.jpg<br>
>><br>
>> 8. two dives: reduce the width of the profile so that the padding on<br>
>> the right of it is equal to the padding bottom from it.<br>
>> see 5.jpg<br>
><br>
><br>
><br>
> I have fixed the issues you have mentioned and I pushed the commits to my<br>
> branch.<br>
><br>
<br>
</span>ok, i have reviewed everything.<br>
<br>
there are still some nitpicks here and there on the bundled templates,<br>
but i will now send the PR and leave it to the community to further<br>
feedback on the project. friday was the "soft pencils down"<br>
date...next week you can finish the documentation and in the meantime<br>
work on the feedback which we are hopefully going to get.<br>
<br>
if there is time you can also complete the "statistics" print (e.g.<br>
support "Edit" for a single statistics template and the color<br>
schemes), but only if it's not going to consume time for the higher<br>
priority tasks.<br>
<br>
21st (friday) is the last date to work on this GSoC project.<br>
<br>
great work so far!<br>
lubomir<br>
--<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">regards,</div></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div><div dir="ltr"><div>Gehad</div></div></div></blockquote></div></div>
</div></div>