help : resizable is not possible

2014-08-17 Thread vavincavent
In the all vue, risizable is not possible because of the icon on the right of graph. So we can see only a little globe. Is it possible to put the icons on the top of the graph? Vavincavent ___ subsurface mailing list subsurface@hohndel.org

Re: help : resizable is not possible

2014-08-17 Thread Dirk Hohndel
On Sun, Aug 17, 2014 at 11:07:13AM +0200, vavincavent wrote: In the all vue, risizable is not possible because of the icon on the right of graph. So we can see only a little globe. Is it possible to put the icons on the top of the graph? I've heard complaints about this from people with small

Re: help : resizable is not possible

2014-08-17 Thread Benjamin
My laptop maxes out at 1366*768. Would it be possible to maybe make the icons scale by a certain percent based on screen size? On 17 Aug 2014 15:25, Dirk Hohndel d...@hohndel.org wrote: On Sun, Aug 17, 2014 at 11:07:13AM +0200, vavincavent wrote: In the all vue, risizable is not possible

Re: help : resizable is not possible

2014-08-17 Thread Tomaz Canabrava
I'll make them Flow on more columns if the size is smaller than the Contents. Em 17/08/2014 10:41, Benjamin nyst...@gmail.com escreveu: My laptop maxes out at 1366*768. Would it be possible to maybe make the icons scale by a certain percent based on screen size? On 17 Aug 2014 15:25, Dirk

Re: help : resizable is not possible

2014-08-17 Thread vavincavent
Dirk, My résolution is 1366x768 with a laptop. Vavincavent Le 17/08/2014 15:25, Dirk Hohndel a écrit : On Sun, Aug 17, 2014 at 11:07:13AM +0200, vavincavent wrote: In the all vue, risizable is not possible because of the icon on the right of graph. So we can see only a little globe. Is it

Re: My progress in HTML export.

2014-08-17 Thread Miika Turkia
Following patch breaks the gas consumption graph: b06b0cd008a76b28f29f6f9fc649ed08e56e6207 HTML: Fix Fake precision value. miika On Sun, Aug 17, 2014 at 1:21 AM, Gehad Elrobey gehadelro...@gmail.com wrote: I attached fixes to some minor bugs that Dirk has mentioned before. Regards,

Gehad's pathes

2014-08-17 Thread Miika Turkia
Hello, yet again a set of patches from Gehad. I did not include the fake precision one as it breaks gas consumption graph. (There is also one patch from me fixing a typo that causes the Location text to be undefined.) miika From ff393e368e615c79df01a99a8b119fd1074c0d9e Mon Sep 17 00:00:00 2001

Re: Partial push of Josh's configure-dive-computer GSoC project

2014-08-17 Thread Thiago Macieira
On Thursday 14 August 2014 10:16:37 Dirk Hohndel wrote: You can conditionally build on the DC_VERSION_CHECK(0,5,0) macro. That would only break if you checkout a much older commit on the master branch. But why would anyone want do that? The master branch is an unstable development branch

[PATCH 1/1] Fix and simplify get_file_name

2014-08-17 Thread Thiago Macieira
This function created a QFile that it didn't need and it returned a dangling pointer (the std::string object was destroyed at the end of the function). The function now returns a pointer that the caller must free. For that reason, it's easier in copy_image_and_overwrite to just go for the

cut and paste

2014-08-17 Thread Dirk Hohndel
I just pushed what I have staged in my master - I need to go through the mailing list one more time, I'm pretty sure I have not picked up all pending patches. But I have a few hours in an airplane ahead of me which sounds like the perfect time to do that. I fixed a few small things, addressed a

Re: cut and paste

2014-08-17 Thread Robert C. Helling
Am 17.08.2014 um 20:12 schrieb Dirk Hohndel d...@hohndel.org: the others who were looking for this kind of functionality... please let me know if this meets your needs or if you need changes in the semantics or UI. Just as an idea from somebody far away from a proper keyboard, let alone

A couple of fixes and updates

2014-08-17 Thread Thiago Macieira
These three patches add to the Fix and simplify get_file_name I sent earlier. ___ subsurface mailing list subsurface@hohndel.org http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface

[PATCH 2/3] Update Subsurface with the updated QSysInfo from Qt 5.4

2014-08-17 Thread Thiago Macieira
The functions changed in Qt since we first backported them to Subsurface. Signed-off-by: Thiago Macieira thi...@macieira.org --- qt-ui/usersurvey.cpp | 39 +++--- subsurfacesysinfo.cpp | 341 +++--- subsurfacesysinfo.h | 17 +-- 3 files changed,

[PATCH 3/3] Use QUrl and QUrlQuery to encode the user survey

2014-08-17 Thread Thiago Macieira
Most of the data we send are simple identifiers, so there wouldn't be a problem. The problem is the free-form user suggestion field, which needs to be properly encoded before it can be sent over the HTTP request. I could have used QUrl even in Qt 5, but using addQueryItem in QUrl in Qt 5 is

[PATCH 1/3] Fix build with Qt 5

2014-08-17 Thread Thiago Macieira
QPrinter is in QtPrintSupport, so we need to declare we want to use it. Signed-off-by: Thiago Macieira thi...@macieira.org --- subsurface.pro | 1 + 1 file changed, 1 insertion(+) diff --git a/subsurface.pro b/subsurface.pro index 0aa6a5d..1d34ee9 100644 --- a/subsurface.pro +++

Re: cut and paste

2014-08-17 Thread Davide DB
Davide@mobile Il 17/ago/2014 20:12 Dirk Hohndel d...@hohndel.org ha scritto: but the biggest change from my side is the cut and paste code. If you hit Ctrl-C (or select Log-Copy dive components) a dialog will open that allows you to select which parts of the currently displayed dive you want

Re: My progress in HTML export.

2014-08-17 Thread Gehad Elrobey
I re-attached the patch and sent two other fixes. On Sun, Aug 17, 2014 at 6:59 PM, Miika Turkia miika.tur...@gmail.com wrote: Following patch breaks the gas consumption graph: b06b0cd008a76b28f29f6f9fc649ed08e56e6207 HTML: Fix Fake precision value. miika On Sun, Aug 17, 2014 at

Re: A couple of fixes and updates

2014-08-17 Thread Dirk Hohndel
On Sun, Aug 17, 2014 at 12:00:57PM -0700, Thiago Macieira wrote: These three patches add to the Fix and simplify get_file_name I sent earlier. Someone from a Comcast internet address in Portland running a Brazilian version of OpenSUSE connected to my server with a broken UserAgent string:

Re: cut and paste

2014-08-17 Thread Dirk Hohndel
On Sun, Aug 17, 2014 at 10:22:19PM +0200, Davide DB wrote: Davide@mobile Il 17/ago/2014 20:12 Dirk Hohndel d...@hohndel.org ha scritto: but the biggest change from my side is the cut and paste code. If you hit Ctrl-C (or select Log-Copy dive components) a dialog will open that allows

Re: My progress in HTML export.

2014-08-17 Thread Miika Turkia
Thanks. Maybe the time unit should be min instead of m as I associate the latter to meters in this context. I just went ahead and changed this in your patch before sending out. miika On Mon, Aug 18, 2014 at 1:08 AM, Gehad Elrobey gehadelro...@gmail.com wrote: I re-attached the patch and sent

Re: A couple of fixes and updates

2014-08-17 Thread Thiago Macieira
On Sunday 17 August 2014 20:34:13 Dirk Hohndel wrote: On Sun, Aug 17, 2014 at 12:00:57PM -0700, Thiago Macieira wrote: These three patches add to the Fix and simplify get_file_name I sent earlier. Someone from a Comcast internet address in Portland running a Brazilian version of OpenSUSE

[PATCH v2 1/1] Update Subsurface with the updated QSysInfo from Qt 5.4

2014-08-17 Thread Thiago Macieira
The functions changed in Qt since we first backported them to Subsurface. Signed-off-by: Thiago Macieira thi...@macieira.org --- qt-ui/usersurvey.cpp | 45 --- subsurfacesysinfo.cpp | 341 +++--- subsurfacesysinfo.h | 17 +-- 3 files changed,

Re: [PATCH v2 1/1] Update Subsurface with the updated QSysInfo from Qt 5.4

2014-08-17 Thread Thiago Macieira
On Sunday 17 August 2014 21:15:13 Thiago Macieira wrote: + // we need to replace all other ':' with ' ' so that this is easy to parse + // in the pretty parts; everything else is an identifier + QString userAgent = QString(Subsurface:%1:%2:%3:%4) +

Re: [PATCH v2 1/1] Update Subsurface with the updated QSysInfo from Qt 5.4

2014-08-17 Thread Dirk Hohndel
On Sun, Aug 17, 2014 at 09:17:45PM -0700, Thiago Macieira wrote: On Sunday 17 August 2014 21:15:13 Thiago Macieira wrote: + // we need to replace all other ':' with ' ' so that this is easy to parse + // in the pretty parts; everything else is an identifier + QString