<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Mar 12, 2017 at 11:41 PM, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Tomaz,<br>
<br>
It's great to see that you are working on things! I had assumed that I'd<br>
have more time this weekend to work on things, but it turned out I got<br>
side tracked with all the pull requests and with fixing the unit system<br>
change bug in the mobile apps...<br>
<span class="gmail-"><br>
On Fri, Mar 10, 2017 at 11:53:16AM +0100, Tomaz Canabrava wrote:<br>
> My time is crazy and I fear I'm taking too long to finish this because I<br>
> just don't find the time to work, sorry.<br>
<br>
</span>I know how that feels.<br>
<span class="gmail-"><br>
> This is the current WIP, there's still a bit of things to be done, I would<br>
> appreciate some help finishing this.<br>
<br>
</span>It would help if you told us what help you need.<br></blockquote><div><span class="gmail-"><br></span></div><div><span class="gmail-"></span></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">
> this is *NOT* a rework like the preferences that added a lot of code for<br>
> little gain, this doesn't adds a lot of code (actually, almost nothing) and<br>
> the gain is big.<br>
<br>
</span>And it would help to understand what that gain is.<br></blockquote><div><br></div><div>The current code holds 6 tabs with all of the logic for saving / updating<br></div><div>restoring a dive, a trip, an dive plan, copy/pasting, etc. all this in one file.<br><br></div><div>This series tries to rework a few things:<br></div><div>- Each tab has it's own header / source, so it's easier to make sure we are actually changing what we wanna change<br></div><div>- Less interconectivity with the MainWindow and other widgets, trying to keep the state more sane.<br></div><div>- Smaller ui files, because a single ui file is massive and sometimes we need to choose strange names on the widges to not have clashes.<br></div><div>- a better, overall, code sanity.<br><br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
The series doesn't apply cleanly for me because my tooling tries to fix<br>
some of the obvious white space garbage as it applies things, and then of<br>
course patches that contain the white space garbage no longer apply.<br>
<br>
It would make my life so much easier if you just used Qt Creator with the<br>
same settings as the rest of us, but I guess that ship has sailed and I<br>
just should stop even asking for that. Which means it might be easier if<br>
you just point me to the branch on github that I can pull from (because<br>
then my tooling won't mess things up).<br></blockquote><div><br></div><div>Sorry - I have a lot of different projects here (work, kde, subsurface), and all them uses a different code style.<br></div><div>The better approach should be QtCreator / KDevelop, whatever, to have different styles per project, but currently it's a no-go.<br></div><div>I'll squash them and use clang-format to have a sane code that you can apply.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Anyway, after I applied the 42 patches things compile and I get a mostly<br>
working UI - but again, I'm not sure what I should be looking at, what I<br>
should be working on.<br></blockquote><div><br></div><div>I'm not changing the code from master, mostly I'm trying to reason with the master code and split it into the different files that I created, so in theory things should still work.<br></div><div>but there are tons of TODOs that I added to the code.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Clearly the Equipment tab and Extra Info tab are non-functional.<br>
Statistics and Photo tab seems unchanged (or identical).<br>
There are small functional issues with the Notes tab (specifically, the<br>
Depth and Duration fields that should only be shown if this is a manually<br>
entered dive are always shown), but other than that this seems reasonably<br>
complete.<br>
<br>
I can easily push this into a branch on github, but unless the rest of us<br>
understand better<br>
<br>
- what are you doing?<br></blockquote><div><br></div><div>Splitting maintab.cpp because it's a mess<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
- why are you doing it (i.e., what's the benefit)?<br></blockquote><div><br></div><div>Smaller functions, better code quality, better to understand what's going on.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
- where would it make sense for us to try to help (so that we aren't<br>
  changing code that you are working on right now)?<br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I'm not quite sure what to do with this.<br></blockquote><div><br> <div>If you could take a look at the tab-widget/ folder I created and 
reason if the changes I'm doing are good, that would be awesome already.<br></div><div>what I need is a "looks fine, continue" - because in my current timeframe I don't wanna work in something that you think it's unecessary.<br><br></div><div>Tomaz<br><br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class="gmail-HOEnZb"><font color="#888888"><br>
/D<br>
<br>
</font></span></blockquote></div><br></div></div>