<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 28, 2015 at 11:18 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">More from Gehad:<br>
- added a "Table" print template - we had this one before<br>
- added a "Flow" print template - as discussed at GSoC start, this one can span<br>
a dive on multiple pages<br>
- Import|Export|Delete of templates - as Dirk suggested<br>
- another bundled color scheme - a blueish one<br>
- more fixes/optimizations here and there<br>
<br>
Feedback is much appreciated at this point! We are reaching the point where<br>
we need to make the bundled templates pretty and write the docs.<br>
<br>
Some comments for Gehad:<br>
<br>
1) I'm getting some new warnings when the app runs initially:<br>
QIODevice::write: device not open<br>
QIODevice::write: device not open<br>
QIODevice::write: device not open<br>
<br>
Are these from the Printing module?<br></blockquote><div><br>I don't get those warnings, I need to check if there is illegal access to the printdevices. <br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
2) Most templates seem so have some sort of white margins at the top and<br>
right.<br>
For instance, due to the offsets in question, it makes the "Table" print cut<br>
the last dive on the current page and the first dive on the next page.<br>
<br>
Do you get these margins on your end? I can send you screenshots if needed.<br></blockquote><div><br>Yes I do, QPrinter adds margins to the pages automatically based on printer settings which corrupts my calculations of the page size, I will work on that issue.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
(BTW, getting the Table print precise in the old code was very tricky and a time<br>
waster; hopefully not the case here.)<br>
<br>
3) I know I suggested to have all the template related buttons (import, export,<br>
etc.) one bellow another, but instead I modified your patch:<br>
Printing: add "Import", "Export" and "Delete" buttons for templates<br>
<br>
to have them 2x2.<br></blockquote><div><br>That's good, I didn't like them one bellow the other, the print dialog was so narrow and tall.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
4) Added one commit from me for safer Import|Export handling if the user<br>
presses "Cancel". This probably needs to be even safer, as we may need to check<br>
if the user-selected file exists.<br>
<br>
5) Slight commit message edits for some of the patches<br>
<br>
6) I was able to solve the cmake/make issue. It could be Windows cmake specific<br>
but to be able to run 'make' after a change without getting an error every<br>
single time, I wrote a script to delete the following folders in the 'build'<br>
folder:<br>
'theme'<br>
'printing_templates'<br>
<br>
Otherwise it just throws "Error: theme/printing_templates is a directory.".<br>
<br>
7) If you are having problems getting the templates the same as we had them<br>
before just send me updates more often so that I pull, check your progress<br>
and comment.<br>
<br></blockquote><div><br>I ll update you as soon as possible.<br><br><br></div></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>