<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 6, 2015 at 11:51 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">hello Gehad,<br>
<span class=""><br>
On 6 August 2015 at 02:25, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
> Hello Lubomir,<br>
><br>
> I wanted to update you with my progress till now, Last week I got stuck with<br>
> a bug that took long time to debug and fix (I didn't fix it completely but<br>
> hopefully will refactor my fixing code soon), here is what I completed till<br>
> now<br>
><br>
> Template enhancement:<br>
> - Manage unused space in (One dive/ Two dives) Templates.<br>
<br>
</span>some of the previous margin issues are still present.<br></blockquote><div><br>The margins issue was due the rendering of the scroll bar which took constant size at the right of the page, I fixed this by hiding the scroll-bar.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> - Added new 6 dives per page template.<br>
<br>
</span>nice, thank you!<br>
<span class=""><br>
> - Remove warning label from dive profile.<br>
<br>
</span>i haven't checked this in the application itself, but the code looks<br>
OK on a quick glance.<br>
<span class=""><br>
><br>
> Fix issues:<br>
> - Handle QColorDialog invalid colors.<br>
> - Fix incorrect state in templateEdit dialog.<br>
> - Remove unnecessary call to Qt5.5 function.<br>
> - Use colors defined in color palette.<br>
><br>
> all commits are pushed to my branch.<br>
><br>
<br>
</span>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></blockquote><div><br>Done.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2) generic changes for *all* templates:<br>
- divide the current border width by two (if by two is too much try * 2/3)<br></blockquote><div><br>Actually there is a problem with sizing the borders in QWebView, I calculate 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 value assigned is always 1px border which appears to be thick in low resolution prints and small paper sizes, while it is so thin in high resolution/ large paper sizes (eg. A0).<br><br>I don't know why "vw" sizing is not working as supposed with border-width in QWebView though it works correctly for me in Chromium & Firefox.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- 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>
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></blockquote><div><br>Done.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2) flow layout:<br>
- there is still some empty space (margin) on the right.<br></blockquote><div><br>Fixed.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- the dive profile is missing - not sure how easy it is to add it?<br></blockquote><div><br>AFAIK there is no straight forward way to implement this, as the flow layout can break dives into pages and breaking the dive profile rendering into pages is not easy, I will put this at my todo list, I can look into it after the scoop of GSoC.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- 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></blockquote><div><br>it was related to the incorrect page margins and fixed.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
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>
4) six dives:<br>
- thanks for adding this one<br>
- there is still some empty space (margin) on the right.<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></blockquote><div><br>The borders issue is still existing, described above.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- the "Table cells 1" color continues on the left and right of the<br>
table, should stop where the outer border ends<br>
- dives at the end of the pages are still cut, probably due to the<br>
unwanted margins on top<br>
<br>
6) two dives:<br>
- there is still some empty space (margin) on the right.<br><br></blockquote><div><br>I guess I fixed all the remaining issues, and pushed them to my branch, please check them and update me if I got any of them incorrectly. I also added a simple statistics view and some other bug fixes.<br><br></div></div>Now I will work mainly on fixing the page scrolling bug and the documentation until you update me with further fixes/enhancements.<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>