diff options
author | Dirk Hohndel <dirk@hohndel.org> | 2013-05-28 09:57:49 -0700 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2013-05-28 09:57:49 -0700 |
commit | 8394828806baf050fa833402c969139d52dc221d (patch) | |
tree | 11f220c56b05030b3b45ec39a0da4b7fd7b41964 | |
parent | b947cc924ff5c78ca7e0561d3ee3891207bea5e6 (diff) | |
download | subsurface-8394828806baf050fa833402c969139d52dc221d.tar.gz |
Fix the positioning of text in the dive profile
This had been bugging me for a while - the label texts were all not
quite where I expected them to be. I think this looks much better now.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
-rw-r--r-- | profile.h | 5 | ||||
-rw-r--r-- | qt-ui/profilegraphics.cpp | 4 |
2 files changed, 5 insertions, 4 deletions
@@ -86,8 +86,9 @@ void setup_pp_limits(struct graphics_context *gc); #define CENTER (-0.5) #define LEFT (0.0) -#define TOP (1) -#define MIDDLE (0) +#define LINE_DOWN (1) +#define TOP (0) +#define MIDDLE (-0.5) #define BOTTOM (-1) #define SCALEXGC(x) (((x) - gc.leftx) / (gc.rightx - gc.leftx) * gc.maxx) diff --git a/qt-ui/profilegraphics.cpp b/qt-ui/profilegraphics.cpp index 722cea6bd..bf244e9e9 100644 --- a/qt-ui/profilegraphics.cpp +++ b/qt-ui/profilegraphics.cpp @@ -332,7 +332,7 @@ void ProfileGraphicsView::plot(struct dive *d, bool forceRedraw) gc.leftx = 0; gc.rightx = 1.0; gc.topy = 0; gc.bottomy = 1.0; - text_render_options_t computer = {DC_TEXT_SIZE, TIME_TEXT, LEFT, MIDDLE}; + text_render_options_t computer = {DC_TEXT_SIZE, TIME_TEXT, LEFT, TOP}; diveComputer = plot_text(&computer, QPointF(gc.leftx, gc.bottomy), nick); // The Time ruler should be right after the DiveComputer: timeMarkers->setPos(0, diveComputer->y()); @@ -926,7 +926,7 @@ void ProfileGraphicsView::plot_depth_profile() timeMarkers = new QGraphicsRectItem(); /* now the text on the time markers */ - struct text_render_options tro = {DEPTH_TEXT_SIZE, TIME_TEXT, CENTER, TOP}; + struct text_render_options tro = {DEPTH_TEXT_SIZE, TIME_TEXT, CENTER, LINE_DOWN}; if (maxtime < 600) { /* Be a bit more verbose with shorter dives */ for (i = incr; i < maxtime; i += incr) |