<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Hey, Lubomir.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
i'm getting some errors:<br>
templatelayout.cpp:310:6: error: redefinition of 'void Dive::put_tags()'<br>
<br>
same for:<br>
::put_sac()<br>
::put_gas()<br>
<br>
removing the duplicates in the .cpp solves the issue.<br>
<br></blockquote><div><br>where are the duplicates? I can't find any.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
i know these layout adjustments are annoying but we need to fix them<br>
before the user feedback as more will stack and we may lose track of<br>
what has to be done.<br>
<span class=""><br>
On 9 August 2015 at 04:10, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
><br>
><br>
> On Thu, Aug 6, 2015 at 11:51 AM, Lubomir I. Ivanov <<a href="mailto:neolit123@gmail.com">neolit123@gmail.com</a>><br>
>><br>
</span><span class="">>> here are some observations and comments. even if i'm not a great<br>
>> designer, i will try to provide you with some detailed feedback on<br>
>> each template. basically i suggest that we solve these before pushing<br>
>> to master and then we can start wrapping things up with community<br>
>> feedback and eventual fixes before the GSoC "pencils down" date.<br>
>><br>
>> 1) all colors themes:<br>
>> i suggest we introduce another color so that we have "Table cells 1"<br>
>> and "Table cells 2". "Table cells 2" will be what color we have at the<br>
>> moment as "Background" and "Background" will be a default white<br>
>> (0xffffff) for all templates and themes. my reasoning behind this is<br>
>> that solid colors on the *whole* page don't make sense for everyone,<br>
>> so some might want to use white, while the second color for the table<br>
>> cells is nice to have.<br>
>><br>
>> also please add a "Table border" color defaulting to black (0x000000)<br>
>> on all templates.<br>
>><br>
><br>
> Done.<br>
<br>
</span>please rename, "Table cells" to "Table cells 1".<br>
<span class=""><br>
><br>
>><br>
>> 2) generic changes for *all* templates:<br>
>> - divide the current border width by two (if by two is too much try * 2/3)<br>
><br>
><br>
> Actually there is a problem with sizing the borders in QWebView, I calculate<br>
> the borders as follows:<br>
><br>
> border-width: max(1px, 0.1vw);<br>
><br>
> For some reasons the view width ratio "0.1vw" is not calculated, so the<br>
> value assigned is always 1px border which appears to be thick in low<br>
> resolution prints and small paper sizes, while it is so thin in high<br>
> resolution/ large paper sizes (eg. A0).<br>
><br>
> I don't know why "vw" sizing is not working as supposed with border-width in<br>
> QWebView though it works correctly for me in Chromium & Firefox.<br>
<br>
</span>"vw" seems to be (used to be) buggy in most browsers.<br>
but having no support for the table border width is quite bad. :\<br>
<br>
what alternatives do we have? can we dynamically adjust the user value<br>
in the templates based on the selected printer DPI or something?<br>
<span class=""><br></span></blockquote><div><br>Yes, we can dynamically adjust the border-size. We can also use Javascript to calculate the needed border-size (I was avoiding to used any javascript code in the templates as I believe it will slow down the rendering). The third option depends mainly on HTML/CSS is to use external div that works as the border to our div, AFAIK "vw" works correctly with div sizes.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
><br>
>><br>
>> - remove the outer most border for all templates except "table". it<br>
>> acts like a separator between dives, but we didn't have it in the<br>
<br>
</span>let's remove the outer border on "table" as well. it just doesn't look<br>
right to have the border's top on the first page and the border's<br>
bottom on the last page.<br>
<span class=""><br>
>> previous version<br>
>> - make the actual data font width regular (not bold) - e.g. Dive No<br>
>> (bold): 232 (not bold)<br>
>> if it looks too small, increase the default font height to 10<br>
>> (probably not needed).<br>
>><br>
><br>
> Done.<br>
><br>
<br>
</span>the font change looks good.<br>
<span class=""><br>
>><br>
>> 2) flow layout:<br>
>> - there is still some empty space (margin) on the right.<br>
><br>
><br>
> Fixed.<br>
><br>
>><br>
>> - the dive profile is missing - not sure how easy it is to add it?<br>
><br>
><br>
> AFAIK there is no straight forward way to implement this, as the flow layout<br>
> can break dives into pages and breaking the dive profile rendering into<br>
> pages is not easy, I will put this at my todo list, I can look into it after<br>
> the scoop of GSoC.<br>
<br>
</span>OK, understood. the idea behind the ideal flow layout would be to<br>
support any length of notes while transitioning dives to new pages,<br>
but also supporting the dive profiles is some way.<br>
<span class=""><br>
><br>
>><br>
>> - the end of last dive on the page is cut and the same applies to the<br>
>> start of the next dive on the new page. is that related to the page<br>
>> calculations or is it a CSS/HTML?<br>
>><br>
><br>
> it was related to the incorrect page margins and fixed.<br>
><br>
<br>
</span>doesn't seem fixed for me. please, see flow.jpg. </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
>><br>
>> 3) one dive:<br>
>> - there is still some empty space (margin) on the right.<br>
>> - it doesn't look like the "one dive per page" in the previous<br>
>> version, but apart from the generic changes we are going to leave it<br>
>> as is, until user feedback<br>
<br>
</span>please, add more padding on the top of the profile so that the<br>
left-right-top padding around the profile is the same.<br>
see 1dive.jpg<br>
<span class=""><br>
>><br>
>> 4) six dives:<br>
>> - thanks for adding this one<br>
>> - there is still some empty space (margin) on the right.<br>
>><br>
<br>
</span>see 6dives.jpg.<br>
there is white space on the top of the page and between the dives.<br>
<span class=""><br>
<br>
>> 5) table:<br>
>> - there are margins on the top, left, right which are undefined. the<br>
>> table should fit the page width and start from the top when no margins<br>
>> are set.<br>
>> - leave the outer border for this template, but reduce it's size as<br>
>> mentioned in "generic"<br>
><br>
><br>
> The borders issue is still existing, described above.<br>
><br>
<br>
</span>as mentioned, remove the outer border for this template as well. make<br>
it so that that the inner borders use the color specified by "Border"<br>
currently it seems that there are no borders around the table cells<br>
and the "Background" color is visible there.<br>
<span class=""><br>
>><br>
>> - the "Table cells 1" color continues on the left and right of the<br>
>> table, should stop where the outer border ends<br>
<br>
</span>ok, this is fixed.<br>
<span class=""><br>
>> - dives at the end of the pages are still cut, probably due to the<br>
>> unwanted margins on top<br>
<br>
</span>i can't reproduce this anymore, but hopefully it won't break for a<br>
different set of dives and different page dimensions.<br>
<span class=""><br>
>><br>
>> 6) two dives:<br>
>> - there is still some empty space (margin) on the right.<br>
>><br>
<br>
</span>see 2dives.jpg<br>
please add some more padding right from the profile.<br>
<br>
at the bottom of this JPG can be seen some white space which is<br>
present on *all* templates. can this be removed?<br>
i think it could be a rounding error when you calculate the page height...<br>
<span class=""><br>
><br>
> I guess I fixed all the remaining issues, and pushed them to my branch,<br>
> please check them and update me if I got any of them incorrectly. I also<br>
> added a simple statistics view and some other bug fixes.<br>
<br>
</span>for the statistics type of print, i think we need use a new set of templates.<br>
so, if the user selects "Statistics" the combo box in "Template" needs<br>
to be re-filled with a different list.<br>
<br>
perhaps a couple of folders in printing_templates need to be used for that:<br>
"divelist"<br>
"statistics"<br>
<br>
in general, i'm not sure how the "Stats" print is going to work. if<br>
it's only going to use one template then we need to disable the Import<br>
/ Export and so on.<br>
if it can support multiple templates then the combo box makes sense, i guess.<br>
<span class=""><br></span></blockquote><div><br>I thought of adding different templates for the statistics view, This will require to export the statistics data structures to Grantlee backend, Which I think will over complicate the printing stack for no actual use-case. So we may need to ask the community for feedback, if adding customized templates for printing statistics is usable feature or not? if not we can disable the combo box and the useless ui elements.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
><br>
> Now I will work mainly on fixing the page scrolling bug and the<br>
> documentation until you update me with further fixes/enhancements.<br>
><br>
<br>
</span>what is the "scrolling bug"?<br>
<br></blockquote><div><br>This bug happens in "table" and "flowlayout" templates, it takes place when the size of the full web view to be rendered is not divisible by the size of the view port (Page size), Please check "scrolling_bug.jpg" for illustration.<br><br>Rendering the web view is completed by scrolling the view port over the web page, after rendering each page QPrinter::newPage() and QWebView:ScrollDown() are called, the incorrect behavior takes place in the last page when the actual scrolled down distance is smaller than the full page size, so there is some area (colored by red in the illustration image) that is rendered twice on the two last pages.</div></div><div><br>I added the rest of your notes to my to do list, and will update you with my progress.<br><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>