Age | Commit message (Collapse) | Author |
|
This makes the code easier to read and manage.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
On macOS the cmake build system doesn't copy the QML resources into the app
bundle and so we do that manually. I forgot to add that for QtCharts.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Trying to keep the different build environments consistent I messed up and
dropped wget and curl from the Coverity build. Moving them to the beginning of
the list so they stand out more.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Sorry about that.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This way the summaries are "more correct".
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Contrary to the Launchpad builds, OBS isn't actually using the spec file that
is included in the Subsurface source but instead maintains its own.
So this is just updating to the version that has been tracked over there,
mostly for reference.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
There are just so many places where I need to remember to update the package
lists. Turns out I had forgotten to add libmtp to the Linux builds when we
add the Garmin Descent mk2 support...
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Several of these stem back from when Subsurface was a Gtk2 app.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
These have all been replaced with the current build process.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
These have all been replaced with the current qmake based build process.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
These haven't been used for a while and still create strange errors.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Pointing at the current build container.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Catching up with all of the changes to Qt installation and our build
processes.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This just makes it more obvious what we are doing.
This appears only necessary on macOS.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
We now require qmake to be found much earlier in the script so we can simply
use that to get the right prefix path.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This was missing in the earlier GitHub Action updates.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Update to Qt 5.12.10, latest OpenSSL, add QtChart, add other missing packages.
Also switch to gcc-7 as our statistics code requires better C++17 support than
what gcc-6 can offer.
This then creates trusty-qt512:1.1
Signed-off-by: Subsurface CI <dirk@hohndel.org>
|
|
Even though the mobile build doesn't have a UI for the statistics, yet, it
already builds the base files.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This fixes another GitHub Action failure.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This simply uses a newer version of our build container which includes newer
tools.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
This hasn't worked in quite a while. It was a good thought, but, yeah, no.
Signed-off-by: Subsurface CI <dirk@hohndel.org>
|
|
To make it clear that the struct tm is only used as an input
parameter.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
Weirdly, this function was declared in dive.h and defined in
subsurface-startup.c. Let's move declaration and definition to
more appropriate places, viz. subsurface-time.h and time.c.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
This function was not used anywhere. Moreover, remove a few
unused includes from qthelper.h. Surprisingly, a number of users
of qthelper.h depend on these, so readd them at the appropriate
places.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
This will be used by the statistics widget.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
one piece of global state removed!
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
Arguably, the number of filtered dives is a matter of the divefilter.
Let's move it there.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
When removing dives, the UndoCommands would keep track of the
shown dives. When adding, they were calling into the filter
instead. Let's remove this asymmetry.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
The shown_dives variable was reset by the dive_list code. Arguably,
the filter should keep track of the number of shown dives, so move
the resetting there. This means adding a new "reset()" member function
to the filter and call that instead of "updateAll()" when the core
data is reset.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
|
|
This appears to be needed only for Android with Qt 5.15. Which means
that this commit creates odd breakage in case someone were to try to
build for Android with an older Qt version - but given that the current
build process only works with Qt 5.14 or 5.15, I think this is an
acceptable flaw.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Can't believe I missed that one.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
The iOS store enforces that every binary needs a new number.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Those are used for the rating / visibility when editing dives.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This seems more intuitive. For editable combo boxes you need to tap on
the indicator, but for non-editable (readonly) ones, you can tap
anywhere and the dropdown is shown.
The code feels a bit clumsy, but seems to work in all cases.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
They always have a 10% darker background, and show a border if the combo
box has focus. This seems to look reasonably well in all situation we
use them.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
It seems that now all elements are correctly re-colored if the user changes
theme.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
With this we should finally get the correct button colors even when
switching color theme.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This one was missed in commit
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Getting the visual right is really hard. The anchors seem to mostly work,
but it still doesn't look exactly right, IMHO.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
Incorrect reference to the Kirigami Units.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
I was convinced that I had fixed this while working on this set of patches,
but apparently I didn't. This simply hardcodes good colors.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This got forgotten earlier.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
I am confused how this worked and then got broken, but this appears to
once again fix the colors.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
hidePassiveNotification() is no longer supported.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
This never ever worked to trigger a profile update. The code is
nonsensical as we cannot access the QMLProfile in a model delegate this
way from outside the delegate.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
If a dive changes, we should simply redraw the profile. This could be
improved by checking for the fields that might impact the profile at
all, but this is definitely a step in the right direction.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|
|
The weird 'Component.onCompleted' always felt like the wrong way to do
this. Setting this directly from the model seems like the much cleaner
solution.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
|