Improve pen & pencil anchor visibility. Fixes bug #986748.

Fixed bugs:
  - https://launchpad.net/bugs/986748

(bzr r11278)

1 files changed, 10 insertions, 10 deletions

diff --git a/src/draw-anchor.cpp b/src/draw-anchor.cpp
index f3a42ca4e..d11ac462f 100644
--- a/src/draw-anchor.cpp
+++ b/src/draw-anchor.cpp
@@ -21,11 +21,13 @@
 #include "display/sodipodi-ctrl.h"
 #include "display/curve.h"
 
+#define FILL_COLOR_NORMAL 0xffffff7f
+#define FILL_COLOR_MOUSEOVER 0xff0000ff
+
 /**
  * Creates an anchor object and initializes it.
  */
-SPDrawAnchor *
-sp_draw_anchor_new(SPDrawContext *dc, SPCurve *curve, gboolean start, Geom::Point delta)
+SPDrawAnchor *sp_draw_anchor_new(SPDrawContext *dc, SPCurve *curve, gboolean start, Geom::Point delta)
 {
     if (SP_IS_LPETOOL_CONTEXT(dc)) {
         // suppress all kinds of anchors in LPEToolContext
@@ -44,8 +46,8 @@ sp_draw_anchor_new(SPDrawContext *dc, SPCurve *curve, gboolean start, Geom::Poin
     a->dp = delta;
     a->ctrl = sp_canvas_item_new(sp_desktop_controls(dt), SP_TYPE_CTRL,
                                  "size", 6.0,
-                                 "filled", 0,
-                                 "fill_color", 0xff00007f,
+                                 "filled", 1,
+                                 "fill_color", FILL_COLOR_NORMAL,
                                  "stroked", 1,
                                  "stroke_color", 0x000000ff,
                                  NULL);
@@ -58,8 +60,7 @@ sp_draw_anchor_new(SPDrawContext *dc, SPCurve *curve, gboolean start, Geom::Poin
 /**
  * Destroys the anchor's canvas item and frees the anchor object.
  */
-SPDrawAnchor *
-sp_draw_anchor_destroy(SPDrawAnchor *anchor)
+SPDrawAnchor *sp_draw_anchor_destroy(SPDrawAnchor *anchor)
 {
     if (anchor->curve) {
         anchor->curve->unref();
@@ -77,21 +78,20 @@ sp_draw_anchor_destroy(SPDrawAnchor *anchor)
  * Test if point is near anchor, if so fill anchor on canvas and return
  * pointer to it or NULL.
  */
-SPDrawAnchor *
-sp_draw_anchor_test(SPDrawAnchor *anchor, Geom::Point w, gboolean activate)
+SPDrawAnchor *sp_draw_anchor_test(SPDrawAnchor *anchor, Geom::Point w, gboolean activate)
 {
     SPDesktop *dt = SP_EVENT_CONTEXT_DESKTOP(anchor->dc);
 
     if ( activate && ( Geom::LInfty( w - dt->d2w(anchor->dp) ) <= A_SNAP ) ) {
         if (!anchor->active) {
-            sp_canvas_item_set((GtkObject *) anchor->ctrl, "filled", TRUE, NULL);
+            sp_canvas_item_set((GtkObject *) anchor->ctrl, "fill_color", FILL_COLOR_MOUSEOVER, NULL);
             anchor->active = TRUE;
         }
         return anchor;
     }
 
     if (anchor->active) {
-        sp_canvas_item_set((GtkObject *) anchor->ctrl, "filled", FALSE, NULL);
+        sp_canvas_item_set((GtkObject *) anchor->ctrl, "fill_color", FILL_COLOR_NORMAL, NULL);
         anchor->active = FALSE;
     }
     return NULL;