Ticket #1902: TargetHighlight.patch

File TargetHighlight.patch, 16.1 KB (added by xeen, 4 years ago)

  • src/org/openstreetmap/josm/actions/mapmode/DrawAction.java 

    Cannot display: file marked as a binary type.
svn:mime-type = application/octet-stream

Property changes on: images\cursor\modifier\joinnode.png
___________________________________________________________________
Name: svn:mime-type
   + application/octet-stream

Cannot display: file marked as a binary type.
svn:mime-type = application/octet-stream

Property changes on: images\cursor\modifier\joinway.png
___________________________________________________________________
Name: svn:mime-type
   + application/octet-stream
     
    6565    private boolean ctrl; 
    6666    private boolean alt; 
    6767    private boolean shift; 
    68     private boolean mouseOnExistingNode; 
     68    private Node mouseOnExistingNode; 
     69    private Set<Way> mouseOnExistingWays = new HashSet<Way>(); 
     70    private Set<OsmPrimitive> oldHighlights = new HashSet<OsmPrimitive>(); 
    6971    private boolean drawHelperLine; 
    7072    private boolean wayIsFinished = false; 
     73    private boolean drawTargetHighlight; 
     74    private boolean drawTargetCursor; 
    7175    private Point mousePos; 
    7276    private Color selectedColor; 
    7377 
    … …  
    9296        return Cursor.getPredefinedCursor(Cursor.CROSSHAIR_CURSOR); 
    9397    } 
    9498 
     99    /** 
     100     * Sets a special cursor to indicate that the next click will automatically join into 
     101     * an existing node or way (if feature is enabled) 
     102     * 
     103     * @param way If true, the cursor will indicate it is joined into a way; node otherwise 
     104     */ 
     105    private void setJoinCursor(boolean way) { 
     106        if(!drawTargetCursor) { 
     107            resetCursor(); 
     108            return; 
     109        } 
     110        try { 
     111            Main.map.mapView.setCursor( 
     112                    ImageProvider.getCursor("crosshair", (way ? "joinway" : "joinnode")) 
     113            ); 
     114            return; 
     115        } catch (Exception e) { 
     116            e.printStackTrace(); 
     117        } 
     118        resetCursor(); 
     119    } 
     120 
     121    /** 
     122     * Resets the cursor to the default cursor for drawing mode (i.e. crosshair) 
     123     */ 
     124    private void resetCursor() { 
     125        Main.map.mapView.setCursor(getCursor()); 
     126    } 
     127 
     128    /** 
     129     * Takes the data from computeHelperLine to determine which ways/nodes should be highlighted 
     130     * (if feature enabled). Also sets the target cursor if appropriate. 
     131     */ 
     132    private void addHighlighting() { 
     133        // if ctrl key is held ("no join"), don't highlight anything 
     134        if (ctrl) { 
     135            removeHighlighting(); 
     136            resetCursor(); 
     137            return; 
     138        } 
     139 
     140        if (mouseOnExistingNode != null) { 
     141            setJoinCursor(false); 
     142            // Clean old highlights 
     143            removeHighlighting(); 
     144            if(drawTargetHighlight) { 
     145                oldHighlights.add(mouseOnExistingNode); 
     146                mouseOnExistingNode.highlighted = true; 
     147            } 
     148            return; 
     149        } 
     150 
     151        // Insert the node into all the nearby way segments 
     152        if(mouseOnExistingWays.size() == 0) { 
     153            removeHighlighting(); 
     154            resetCursor(); 
     155            return; 
     156        } 
     157 
     158        setJoinCursor(true); 
     159        // Clean old highlights 
     160        removeHighlighting(); 
     161 
     162        if(!drawTargetHighlight) return; 
     163        oldHighlights.addAll(mouseOnExistingWays); 
     164        for(Way w : mouseOnExistingWays) { 
     165            w.highlighted = true; 
     166        } 
     167    } 
     168 
     169    /** 
     170     * Removes target highlighting from primitives 
     171     */ 
     172    private void removeHighlighting() { 
     173        for(OsmPrimitive prim : oldHighlights) { 
     174            prim.highlighted = false; 
     175        } 
     176    } 
     177 
    95178    @Override public void enterMode() { 
    96179        super.enterMode(); 
    97180        selectedColor = Main.pref.getColor(marktr("selected"), Color.red); 
    98181        drawHelperLine = Main.pref.getBoolean("draw.helper-line", true); 
     182        drawTargetHighlight = Main.pref.getBoolean("draw.target-highlight", true); 
     183        drawTargetCursor = Main.pref.getBoolean("draw.target-cursor", true); 
    99184        wayIsFinished = false; 
    100          
     185 
    101186        Main.map.mapView.addMouseListener(this); 
    102187        Main.map.mapView.addMouseMotionListener(this); 
    103188        Main.map.mapView.addTemporaryLayer(this); 
    104189        DataSet.selListeners.add(this); 
     190 
    105191        try { 
    106192            Toolkit.getDefaultToolkit().addAWTEventListener(this, AWTEvent.KEY_EVENT_MASK); 
    107193        } catch (SecurityException ex) { 
    … …  
    109195        // would like to but haven't got mouse position yet: 
    110196        // computeHelperLine(false, false, false); 
    111197    } 
     198 
    112199    @Override public void exitMode() { 
    113200        super.exitMode(); 
    114201        Main.map.mapView.removeMouseListener(this); 
    115202        Main.map.mapView.removeMouseMotionListener(this); 
    116203        Main.map.mapView.removeTemporaryLayer(this); 
    117204        DataSet.selListeners.remove(this); 
     205        removeHighlighting(); 
    118206        try { 
    119207            Toolkit.getDefaultToolkit().removeAWTEventListener(this); 
    120208        } catch (SecurityException ex) { 
    … …  
    146234        Main.ds.setSelected(); 
    147235        mouseClicked(e); 
    148236    } 
    149      
     237 
    150238    /** 
    151239     * If user clicked with the left button, add a node at the current mouse 
    152240     * position. 
    … …  
    183271 
    184272        if (!ctrl) 
    185273            n = Main.map.mapView.getNearestNode(mousePos); 
    186        
     274 
    187275        if (n != null) { 
    188276            // user clicked on node 
    189277            if (shift || selection.isEmpty()) { 
    … …  
    258346        if (!shift && selection.size() > 0 && !wayIsFinishedTemp) { 
    259347            Node selectedNode = null; 
    260348            Way selectedWay = null; 
    261              
     349 
    262350            for (OsmPrimitive p : selection) { 
    263351                if (p instanceof Node) { 
    264352                    if (selectedNode != null) { 
    … …  
    276364                    selectedWay = (Way) p; 
    277365                } 
    278366            } 
    279              
     367 
    280368            // No nodes or ways have been selected, try again with no selection 
    281369            // This occurs when a relation has been selected 
    282370            if(selectedNode == null && selectedWay == null) { 
    … …  
    291379                if (selectedWay.isFirstLastNode(lastUsedNode)) { 
    292380                    n0 = lastUsedNode; 
    293381                } else { 
    294                     // We have a way selected, but no suitable node to continue from. Start anew.  
     382                    // We have a way selected, but no suitable node to continue from. Start anew. 
    295383                    tryAgain(e); 
    296384                    return; 
    297385                } 
    … …  
    325413                    return; 
    326414                } 
    327415            } 
    328              
     416 
    329417            // User clicked last node again, finish way 
    330418            if(n0 == n) { 
    331419                lastUsedNode = null; 
    … …  
    378466            extendedWay = true; 
    379467            Main.ds.setSelected(way); 
    380468        } 
    381        
     469 
    382470        String title; 
    383471        if (!extendedWay) { 
    384472            if (!newNode) { 
    … …  
    403491        Main.main.undoRedo.add(c); 
    404492        if(!wayIsFinished) lastUsedNode = n; 
    405493        computeHelperLine(); 
     494        removeHighlighting(); 
    406495        Main.map.mapView.repaint(); 
    407496    } 
    408497 
    … …  
    419508        shift = (e.getModifiers() & ActionEvent.SHIFT_MASK) != 0; 
    420509        mousePos = e.getPoint(); 
    421510 
     511        addHighlighting(); 
    422512        computeHelperLine(); 
    423513    } 
    424514 
    … …  
    443533        Node selectedNode = null; 
    444534        Way selectedWay = null; 
    445535        Node currentMouseNode = null; 
    446         mouseOnExistingNode = false; 
     536        mouseOnExistingNode = null; 
     537        mouseOnExistingWays = new HashSet<Way>(); 
    447538 
    448539        Main.map.statusLine.setAngle(-1); 
    449540        Main.map.statusLine.setHeading(-1); 
    … …  
    453544            currentMouseNode = Main.map.mapView.getNearestNode(mousePos); 
    454545        } 
    455546 
     547        // We need this for highlighting and we'll only do so if we actually want to re-use 
     548        // *and* there is no node nearby (because nodes beat ways when re-using) 
     549        if(!ctrl && currentMouseNode == null) { 
     550            List<WaySegment> wss = Main.map.mapView.getNearestWaySegments(mousePos); 
     551            for(WaySegment ws : wss) 
     552                mouseOnExistingWays.add(ws.way); 
     553        } 
     554 
    456555        if (currentMouseNode != null) { 
    457556            // user clicked on node 
    458557            if (selection.isEmpty()) return; 
    459558            currentMouseEastNorth = currentMouseNode.eastNorth; 
    460             mouseOnExistingNode = true; 
     559            mouseOnExistingNode = currentMouseNode; 
    461560        } else { 
    462561            // no node found in clicked area 
    463562            currentMouseEastNorth = Main.map.mapView.getEastNorth(mousePos.x, mousePos.y); 
    … …  
    510609        Main.map.statusLine.setDist(distance); 
    511610        updateStatusLine(); 
    512611 
    513         if (!drawHelperLine || wayIsFinished) return; 
    514  
     612        if ((!drawHelperLine || wayIsFinished) && !drawTargetHighlight) return; 
    515613        Main.map.mapView.repaint(); 
    516614    } 
    517615 
    … …  
    639737    } 
    640738 
    641739    public void paint(Graphics g, MapView mv) { 
     740        if (!drawHelperLine && !drawTargetHighlight) return; 
    642741 
    643         // don't draw line if disabled in prefs 
    644         if (!drawHelperLine) return; 
    645  
    646742        // sanity checks 
    647743        if (Main.map.mapView == null) return; 
    648744        if (mousePos == null) return; 
    … …  
    651747        if (shift) return; 
    652748 
    653749        // don't draw line if we don't know where from or where to 
    654         if (currentBaseNode == null) return; 
    655         if (currentMouseEastNorth == null) return; 
     750        if (currentBaseNode == null || currentMouseEastNorth == null) return; 
    656751 
    657752        // don't draw line if mouse is outside window 
    658753        if (!Main.map.mapView.getBounds().contains(mousePos)) return; 
    … …  
    676771 
    677772        g2.draw(b); 
    678773        g2.setStroke(new BasicStroke(1)); 
    679  
    680774    } 
    681775 
    682776    @Override public String getModeHelpText() { 
    683777        String rv; 
    684778 
    685779        if (currentBaseNode != null && !shift) { 
    686             if (mouseOnExistingNode) { 
     780            if (mouseOnExistingNode != null) { 
    687781                if (alt && /* FIXME: way exists */true) 
    688782                    rv = tr("Click to create a new way to the existing node."); 
    689783                else 
    … …  
    703797        //rv.append(tr("Click to add a new node. Ctrl: no node re-use/auto-insert. Shift: no auto-connect. Alt: new way")); 
    704798        return rv.toString(); 
    705799    } 
    706      
     800 
    707801    @Override public boolean layerIsSupported(Layer l) { 
    708802        return l instanceof OsmDataLayer; 
    709803    } 

  • src/org/openstreetmap/josm/data/osm/OsmPrimitive.java

     
    122122     * If set to true, this object is currently selected. 
    123123     */ 
    124124    public volatile boolean selected = false; 
     125     
     126    /** 
     127     * If set to true, this object is highlighted. Currently this is only used to 
     128     * show which ways/nodes will connect  
     129     */ 
     130    public volatile boolean highlighted = false; 
    125131 
    126132    /** 
    127133     * Time of last modification to this object. This is not set by JOSM but 

  • src/org/openstreetmap/josm/data/osm/visitor/MapPaintVisitor.java

     
    119119 
    120120        if(osm.mappaintStyle == null && styles != null) { 
    121121            osm.mappaintStyle = styles.get(osm); 
    122             if(osm instanceof Way) 
    123                 osm.isMappaintArea = styles.isArea(osm); 
     122            osm.isMappaintArea = styles.isArea(osm); 
    124123        } 
    125124        return osm.isMappaintArea; 
    126125    } 
    … …  
    150149 
    151150        if (nodeStyle != null && isZoomOk(nodeStyle) && showIcons > dist) 
    152151            drawNode(n, nodeStyle.icon, nodeStyle.annotate, n.selected); 
     152        else if (n.highlighted) 
     153            drawNode(n, highlightColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode); 
    153154        else if (n.selected) 
    154155            drawNode(n, selectedColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode); 
    155156        else if (n.tagged) 
    … …  
    265266            int tmpWidth = (int) (100 /  (float) (circum / realWidth)); 
    266267            if (tmpWidth > width) width = tmpWidth; 
    267268        } 
    268         if(w.selected) 
     269 
     270        if(w.highlighted) 
     271            color = highlightColor; 
     272        else if(w.selected) 
    269273            color = selectedColor; 
    270274 
    271275        // draw overlays under the way 
    … …  
    11351139                } 
    11361140                g2d.draw(currentPath); 
    11371141            } 
    1138              
     1142 
    11391143            if(useStrokes > dist) 
    11401144                g2d.setStroke(new BasicStroke(1)); 
    11411145 

  • src/org/openstreetmap/josm/data/osm/visitor/SimplePaintVisitor.java

     
    3636    public final static Color darkblue = new Color(0,0,128); 
    3737    public final static Color darkgreen = new Color(0,128,0); 
    3838    public final static Color teal = new Color(0,128,128); 
     39    public final static Color lightteal= new Color(0, 255, 186); 
    3940 
    4041    /** 
    4142     * The environment to paint to. 
    … …  
    6162    protected Color untaggedWayColor; 
    6263    protected Color incompleteColor; 
    6364    protected Color backgroundColor; 
     65    protected Color highlightColor; 
    6466    protected boolean showDirectionArrow; 
    6567    protected boolean showRelevantDirectionsOnly; 
    6668    protected boolean showHeadArrowOnly; 
    … …  
    9698        untaggedWayColor = Main.pref.getColor(marktr("untagged way"), darkgreen); 
    9799        incompleteColor = Main.pref.getColor(marktr("incomplete way"), darkerblue); 
    98100        backgroundColor = Main.pref.getColor(marktr("background"), Color.BLACK); 
     101        highlightColor = Main.pref.getColor(marktr("highlight"), lightteal); 
    99102    } 
    100103 
    101104    protected void getSettings(Boolean virtual) { 
    … …  
    131134            System.out.println("Simplepaint Profiler"); 
    132135 
    133136        getSettings(virtual); 
    134          
    135         if(profiler)  
     137 
     138        if(profiler) 
    136139        { 
    137140            System.out.format("Prepare  : %4dms\n", (java.lang.System.currentTimeMillis()-profilerLast)); 
    138141            profilerLast = java.lang.System.currentTimeMillis(); 
    139142        } 
    140          
     143 
    141144        // draw tagged ways first, then untagged ways. takes 
    142145        // time to iterate through list twice, OTOH does not 
    143146        // require changing the colour while painting... 
    … …  
    244247 
    245248        if (inactive) 
    246249            drawNode(n, inactiveColor, unselectedNodeSize, unselectedNodeRadius, fillUnselectedNode); 
     250        else if (n.highlighted) 
     251            drawNode(n, highlightColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode); 
    247252        else if (n.selected) 
    248253            drawNode(n, selectedColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode); 
    249254        else if(n.tagged) 
    … …  
    300305 
    301306        if (inactive) { 
    302307            wayColor = inactiveColor; 
     308        } else if(w.highlighted) { 
     309            wayColor = highlightColor; 
     310        } else if(w.selected) { 
     311            wayColor = selectedColor; 
    303312        } else if (!w.tagged) { 
    304313            wayColor = untaggedWayColor; 
    305314        } else { 
    … …  
    311320            Point lastP = nc.getPoint(it.next().eastNorth); 
    312321            for (int orderNumber = 1; it.hasNext(); orderNumber++) { 
    313322                Point p = nc.getPoint(it.next().eastNorth); 
    314                 drawSegment(lastP, p, w.selected && !inactive ? selectedColor : wayColor,  
     323                drawSegment(lastP, p, wayColor, 
    315324                    showOnlyHeadArrowOnly ? !it.hasNext() : showThisDirectionArrow); 
    316325                if (showOrderNumber) 
    317326                    drawOrderNumber(lastP, p, orderNumber);