<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 27, 2015 at 8:31 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 27 July 2015 at 18:10, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
> Hello Lubomir,<br>
><br>
> This week I was working on the following tasks:<br>
><br>
> - Search for templates in printing_templates directory.<br>
> - Fix QSettingsGroup issue.<br>
> - Make TemplateEdit dialog re-sizable.<br>
> - Implement import,export and delete templates.<br>
> - Add shades of blue color palette.<br>
> - Add flow layout.<br>
> - Add table layout.<br>
><br>
<br>
</span>hello Gehad,<br>
nicely done!<br>
<br>
here are some quick comments:<br>
<br>
1) "Printing: get the number of pages from the full resolution"<br>
(cosmetic) this patch has redundant braces { } around a single lined<br>
if/then statement.<br>
<br>
2) "Printing: remove existing values in print template combobox"<br>
it might be a good idea to rename all bundled templates to have spaces<br>
and first letter capitalization in the filenames - e.g.:<br>
"One dive"<br>
"Two dives"<br>
"Table"<br>
etc.<br>
<br>
3) "Printing: support importing/exporting of templates"<br>
i think users will find the extra window menu (Template->Import |<br>
Export | Delete) a bit confusing.<br>
<br>
please, move the actions as buttons in the already present "Template" QGroupBox.<br>
attached is a mockup of how i think it should look like. enjoy my<br>
amazing drawing "skills".<br>
<br>
4) one commit message related advice for future cases...<br>
it's common practice not to personalize sentences in official text<br>
documents. this includes the Subsurface commit messages as a non-super<br>
strict rule (TMK), but in general it's a very *clean* thing to have.<br>
<br>
real world examples can range from everything in law papers to academia theses.<br>
<br>
a self-explanatory GIT commit message example would be:<br>
"Feature <X> was added in patch <SHA1>, but..."<br>
instead of:<br>
"User <Y> added feature <X> was added in patch <SHA1>, but..."<br>
or:<br>
"We/I/He/She have/has added feature <X> in patch <SHA1>, but..."<br>
<span class=""><br>
> I am still working on the following tasks:<br>
> - Enhancing the templates.<br>
> - Don't break dives into pages while using table/flow layout.<br>
><br>
<br>
</span>yeah, once these patches are pushed, having the templates as close as<br>
what we had before becomes our highest priority.<br>
<br>
lubomir<br>
--<br>
</blockquote></div><br clear="all"><div>I pushed the updates to my branch.<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>