aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)Author
2013-12-07Make units in cells consistant in CylindersModelGravatar Anton Lundin
Now they all have units in the cells. Fixes #259 Signed-off-by: Anton Lundin <glance@acc.umu.se> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-07amount_selected should never be decremented if already 0Gravatar Martin Gysel
this is probably more a workaround than a proper fix to a problem which I think only happens if started with no dive loaded and then create one. in that case, deselect_dive is called after saving the dive but probalby before it was added to the dive list properly resulting in an underflow of amount_selected. Signed-off-by: Martin Gysel <me@bearsh.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-07redraw profile (actually clean it) if no dive is selectedGravatar Martin Gysel
otherwise the previous selected dive is still visible in the profile view. clicking on edit crashs subsurface: null pointer dereference in editCurrentDive as it uses 'current_dive' which is null. Signed-off-by: Martin Gysel <me@bearsh.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-07correctly install translations after a clean checkoutGravatar Martin Gysel
Signed-off-by: Martin Gysel <me@bearsh.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-07Fix profile legend stringsGravatar Sergey Starosek
Use proper case and subscripts for gas names on profile legend. Signed-off-by: Sergey Starosek <sergey.starosek@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-07Fixing a few typos in DocumentationGravatar Werner Macho
Signed-off-by: Werner Macho <werner.macho@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-07Renumber dialog should default to start at 1Gravatar Dirk Hohndel
Also make it somewhat larger and less crowded. Fixes #338 Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Get ready for the release of 3.9.2v3.9.2Gravatar Dirk Hohndel
Small updates to README, ReleaseNotes.txt and a bump in the version number in the Makefile (forgot the in 3.9.1 - thankfully this is automatic for git based builds). Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Set the menuRole property on certain menu entry QActionsGravatar Thiago Macieira
Telling Qt that the "Preferences" menu entry is QAction::PreferencesRole will make it be moved to the System menu on Mac, for example. Fixes #327 Signed-off-by: Thiago Macieira <thiago@macieira.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Fix the Windwos installerGravatar Dirk Hohndel
Install the Documentation and include it in the installer. Try and get all the directories and files removed in the uninstaller. Where the heck does 'oldshare' come from? Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Correctly parse translated GPS coordinatesGravatar Dirk Hohndel
Minor oversight in commit 917b47b79cb8 ("Parse localized GPS string") - we accepted the translated hemisphere indicators in the regular expression, but then didn't use them later when comparing with the result of applying the regular expression. Fixes #331 Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Really save airtemp and watertemp in the diveGravatar Dirk Hohndel
I thought I had this fixed, but a silly thinko broke my fix. Only abort the saving if both air AND water temperature are already covered by the divecomputer(s) in a dive. If at least one of them is different, then create a divetemperature tag and fill in that value at the dive level. Fixes #313 Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Stub out a mark dive invalid functionGravatar Dirk Hohndel
Not sure I'll be able to finish this for 4.0 Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Finish download from divelogs.deGravatar Dirk Hohndel
This is far from perfect. Avoid a weird warning when total = -1 is passed into the progress bar. Enable the Accept button once the download completes. Merge the downloaded dives into the existing dive list on Apply. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Whitespace cleanupGravatar Dirk Hohndel
That one slipped by me... Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Do not change the current DirectoryGravatar Tomaz Canabrava
Qt has a really strange API for directories. when you create a QDir and set's the currentDir on *that* QDir you created, it's for the whole application and not just for that variable. Then when the variable is out of scope, the directory doesn't change back - ugh. The solution is not to call setCurrentDir on the newly created QDir (which was quite good since it's also a code cleanup. ) [Dirk Hohndel: modified to undo the far less elegant fix I had made in an earlier commit] Signed-off-by: Tomaz Canabrava <tcanabrava@kde.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Latest translationsGravatar Dirk Hohndel
As usual, only (some of) the changes to the German translations were authored by me. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Latest source stringsGravatar Dirk Hohndel
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Don't change working directory when looking for tranlationsGravatar Dirk Hohndel
And only replace 'bin' if there is a bin in the path... Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Added 'English' and 'Use System Default' options.Gravatar Tomaz Canabrava
These complete the ability to select languages from the preferences panel. Signed-off-by: Tomaz Canabrava <tcanabrava@kde.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Added a filter option to the Language ChooserGravatar Tomaz Canabrava
This patch adds a filter option to the Language Chooser. if you choose filter, only the filtered options will appear. Signed-off-by: Tomaz Canabrava <tcanabrava@kde.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Explain invalid dive option in preferencesGravatar Dirk Hohndel
Just trying to fill in minor gaps. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Capitalize Subsurface in the documentationGravatar Dirk Hohndel
This is a small detail but we should try to be consistent there. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Add images missing from previous commitGravatar Willem Ferguson
This was Dirk's mistake when applying the patch. Signed-off-by: Willem Ferguson <willemferguson@zoology.up.ac.za> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Usermanual update.Gravatar Willem Ferguson
1) Several layout changes including banner. 2) Stylistic revision of part of textr. 3) Addition of text to dive computer upload section. Signed-off-by: Willem Ferguson <willemferguson@zoology.up.ac.za> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Code CleanupGravatar Tomaz Canabrava
Signed-off-by: Tomaz Canabrava <tcanabrava@kde.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Settings for Display preferences didn't really set the preferences.Gravatar Tomaz Canabrava
The code that should set the preferences was actually retrieving it, this will correctly set it. Signed-off-by: Tomaz Canabrava <tcanabrava@kde.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Added a language preference to the Settings.Gravatar Tomaz Canabrava
When the user first opens the application the default language is selected; this can be changed to a hardcoded one by going to system preferences and choosing the one you want. Restart required. Fixes #136 [Dirk Hohndel: whitespace fixes, removed qDebug() call, rephrased the message displayed prompting the user to restart.] Signed-off-by: Tomaz Canabrava <tcanabrava@kde.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Print: add a safe-guard if table print rows are too bigGravatar Lubomir I. Ivanov
We do not support rows larger than a single page as the PrintLayout algorithm will enter an infinite loop, therefore we put a limit of 15 lines for the location text. Signed-off-by: Lubomir I. Ivanov <neolit123@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Fix crash in table print crashGravatar Dirk Hohndel
sizeof(array) is the total size in bytes, not the number of elements... Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Print: fix issues when printing a lot of dives in table printGravatar Lubomir I. Ivanov
This patch improves the algorithm when estimating where to put the new page header in the table and how we move larger dive rows on a new page. It now performs a couple of 'passes', where the first one processes the table and the second one is used to compensate for the lost space. Fixes #326 Signed-off-by: Lubomir I. Ivanov <neolit123@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Companion app and downloading GPS coordinates from web serviceGravatar Salvador Cuñat
A first try on documenting the companion app and explanation on how to use it combined with Subsurface. Signed-off-by: Salvador Cuñat <salvador.cunat@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-06Add icon and title for User Manual windowGravatar Sergey Starosek
Signed-off-by: Sergey Starosek <sergey.starosek@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05Latest translation updatesGravatar Dirk Hohndel
As alsway, I am not the author of these translations, just the committer. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05Add more languagesGravatar Dirk Hohndel
Not all of these are completed, but I want to include as many as possible in the last beta. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05Include the translations in the Windows installerGravatar Thiago Macieira
Fixes #324 Signed-off-by: Thiago Macieira <thiago@macieira.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05Install Qt and Subsurface translations for WindowsGravatar Thiago Macieira
Dirk had done most of the work already. I just needed to add some trickery to install the system translations in case cross-mingw32 didn't provide it. Signed-off-by: Thiago Macieira <thiago@macieira.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05Install Subsurface translation files on LinuxGravatar Thiago Macieira
This was the easiest: all the file names are already in a variable and we don't need to install the Qt translations since we're using a system Qt. Signed-off-by: Thiago Macieira <thiago@macieira.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05Fix typo introduced accidentally: marbledir has no hGravatar Thiago Macieira
Signed-off-by: Thiago Macieira <thiago@macieira.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05Avoid copying the icons, xslt and other dirs into themselves.Gravatar Thiago Macieira
The install rule that qmake generates was of the form: -$(INSTALL_DIR) icons packaging/windows/icons The first time you run make install, packaging/windows/icons doesn't exist, so /usr/bin/install understands it as the name to be given to the dir being installed. However, when you run make install a second time, /usr/bin/install understands it as the target directory, so it copies icons into the packaging/windows/icons dir. We need to teach qmake not to add the dir name. Inspecting the source code reveals: if(fi.isDir() && project->isActiveConfig("copy_dir_files")) { if(!dst_file.endsWith(Option::dir_sep)) dst_file += Option::dir_sep; dst_file += fi.fileName(); } This option is enabled by default on all the win32-* targets. So if we remove the "copy_dir_files" config, it won't append the source name. The qmake manual says this about that option: "Enables the install rule to also copy directories, not just files.", which doesn't help us much. Probably just stale documentation. Fixes #325 Signed-off-by: Thiago Macieira <thiago@macieira.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05Updates to user manualGravatar Willem Ferguson
1) Inserted section on Information Box on dive profile 2) Iserted short section on import from Mares Dive Organiser 3) Corrections to spelling and style not included in previous commits. Signed-off-by: Willem Ferguson <willemferguson@zoology.up.ac.za> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05MainWindow: fix setTitle() callsGravatar Lubomir I. Ivanov
setTitle() works with an enum that can be fed different values, and other we may have eventually. For example not only "app title", or "app title: filename", but others, like dates, certain app status and so on. Patch fixes an issue where deleting a single dive in the divelist resets the title to "Subsurface" only. clearUpEmpty() should only call setTitle(MWTF_DEFAULT) if no file is currently open. It also adds a safe-guard to revert back to MWTF_DEFAULT if no file is open but setTitle is called with MWTF_FILENAME. Signed-off-by: Lubomir I. Ivanov <neolit123@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05Print: fix truncated table in "Table print" modeGravatar Lubomir I. Ivanov
The whole procedure here is quite confusing. Once we have our model populated, we need to estimate where to put page headers (each page has one) and to do that we store a list of indexes, where a page would begin (pageIndexes). But since a row can end up being chopped at the end of a page we move it to a new page and this particular part was lacking compensation for the moved row's height, when storing the *last* pageIndex. For N number of pages we are losing N - 1 dives, or such that were previously transferred on a new page. See #326 Signed-off-by: Lubomir I. Ivanov <neolit123@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05Open external links in default browserGravatar Sergey Starosek
Signed-off-by: Sergey Starosek <sergey.starosek@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-05User manual spelling fixesGravatar Sergey Starosek
Signed-off-by: Sergey Starosek <sergey.starosek@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-04Different Icon for a Bookmark Event.Gravatar Tomaz Canabrava
This patch adds a different icon for a Bookmark Event, and it also cleaned a lot of code. :) See #300 [Dirk Hohndel: made the two icons slightly bigger] Signed-off-by: Tomaz Canabrava <tcanabrava@kde.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-04Fixes positioning of the legend in View and Print modes.Gravatar Tomaz Canabrava
This patch makes a better assumption of the location for the legend in both View and Print modes. It also fixes a few oddities that we used to have ( like hardcoded spacing ). We are taking the scene().sceneRect() now into consideration to better place it on canvas. Fixes: #322 Signed-off-by: Tomaz Canabrava <tcanabrava@kde.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-04Compile fixGravatar Dirk Hohndel
Otherwise errno is undefined (at least for me on Linux). Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-12-04Merge branch 'improve-subsurfaceweb' of ↵Gravatar Dirk Hohndel
git://github.com/thiagomacieira/subsurface
2013-12-04Cleanup some uninitialized variablesGravatar Anton Lundin
I can't really see any point in passing a local loop variable around, and copying a uninitialized pointer. Better use local variables there and let the compiler optimize them away if it feels for doing that. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>