Changeset 7608 in josm for trunk


Ignore:
Timestamp:
2014-10-08T00:01:23+02:00 (10 years ago)
Author:
Don-vip
Message:

fix #10554 - Add notes dialog (patch by ToeBee)

Location:
trunk
Files:
10 added
1 deleted
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadNotesTask.java

    r7597 r7608  
    9696            if (noteLayers != null && noteLayers.size() > 0) {
    9797                layer = noteLayers.get(0);
    98                 layer.addNotes(notesData);
     98                layer.getNoteData().addNotes(notesData);
    9999            } else {
    100100                layer = new NoteLayer(notesData, "Notes");

  • trunk/src/org/openstreetmap/josm/gui/MapFrame.java

    r7483 r7608  
    6767import org.openstreetmap.josm.gui.dialogs.LayerListDialog;
    6868import org.openstreetmap.josm.gui.dialogs.MapPaintDialog;
     69import org.openstreetmap.josm.gui.dialogs.NoteDialog;
    6970import org.openstreetmap.josm.gui.dialogs.RelationListDialog;
    7071import org.openstreetmap.josm.gui.dialogs.SelectionListDialog;
    … …  
    133134    public SelectionListDialog selectionListDialog;
    134135    public PropertiesDialog propertiesDialog;
     136    public NoteDialog noteDialog;
    135137
    136138    // Map modes
    … …  
    243245        addToggleDialog(new ChangesetDialog(), true);
    244246        addToggleDialog(new MapPaintDialog());
     247        //TODO: remove this if statement once note support is complete
     248        if(Main.pref.getBoolean("osm.notes.enableDownload", false)) {
     249            addToggleDialog(noteDialog = new NoteDialog());
     250        }
    245251        toolBarToggle.setFloatable(false);
    246252

  • trunk/src/org/openstreetmap/josm/gui/layer/NoteLayer.java

    r7522 r7608  
    44import static org.openstreetmap.josm.tools.I18n.tr;
    55
     6import java.awt.Dimension;
    67import java.awt.Graphics2D;
    78import java.awt.Point;
     9import java.awt.event.MouseEvent;
     10import java.awt.event.MouseListener;
     11import java.text.SimpleDateFormat;
    812import java.util.ArrayList;
    913import java.util.List;
    … …  
    1216import javax.swing.Icon;
    1317import javax.swing.ImageIcon;
     18import javax.swing.JToolTip;
    1419
    1520import org.openstreetmap.josm.Main;
    … …  
    1823import org.openstreetmap.josm.data.notes.Note.State;
    1924import org.openstreetmap.josm.data.notes.NoteComment;
     25import org.openstreetmap.josm.data.osm.NoteData;
    2026import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
    2127import org.openstreetmap.josm.gui.MapView;
    2228import org.openstreetmap.josm.gui.dialogs.LayerListDialog;
    2329import org.openstreetmap.josm.gui.dialogs.LayerListPopup;
    24 import org.openstreetmap.josm.tools.ImageProvider;
     30import org.openstreetmap.josm.gui.dialogs.NoteDialog;
     31import org.openstreetmap.josm.io.XmlWriter;
     32import org.openstreetmap.josm.tools.ColorHelper;
    2533
    2634/**
    2735 * A layer to hold Note objects
    2836 */
    29 public class NoteLayer extends AbstractModifiableLayer {
    30 
    31     private final List<Note> notes;
     37public class NoteLayer extends AbstractModifiableLayer implements MouseListener {
     38
     39    private final NoteData noteData;
    3240
    3341    /**
    … …  
    3846    public NoteLayer(List<Note> notes, String name) {
    3947        super(name);
    40         this.notes = notes;
     48        noteData = new NoteData(notes);
     49        init();
     50    }
     51
     52    /** Convenience constructor that creates a layer with an empty note list */
     53    public NoteLayer() {
     54        super(tr("Notes"));
     55        noteData = new NoteData();
     56        init();
     57    }
     58
     59    private void init() {
     60        if (Main.map != null && Main.map.mapView != null) {
     61            Main.map.mapView.addMouseListener(this);
     62        }
     63    }
     64
     65    /**
     66     * Returns the note data store being used by this layer
     67     * @return noteData containing layer notes
     68     */
     69    public NoteData getNoteData() {
     70        return noteData;
    4171    }
    4272
    4373    @Override
    4474    public boolean isModified() {
    45         for (Note note : notes) {
     75        for (Note note : noteData.getNotes()) {
    4676            if (note.getId() < 0) { //notes with negative IDs are new
    4777                return true;
    … …  
    6393    @Override
    6494    public void paint(Graphics2D g, MapView mv, Bounds box) {
    65         for (Note note : notes) {
     95        for (Note note : noteData.getNotes()) {
    6696            Point p = mv.getPoint(note.getLatLon());
    6797
    6898            ImageIcon icon = null;
    6999            if (note.getId() < 0) {
    70                 icon = ImageProvider.get("notes", "note_new_16x16.png");
     100                icon = NoteDialog.ICON_NEW_SMALL;
    71101            } else if (note.getState() == State.closed) {
    72                 icon = ImageProvider.get("notes", "note_closed_16x16.png");
     102                icon = NoteDialog.ICON_CLOSED_SMALL;
    73103            } else {
    74                 icon = ImageProvider.get("notes", "note_open_16x16.png");
     104                icon = NoteDialog.ICON_OPEN_SMALL;
    75105            }
    76106            int width = icon.getIconWidth();
    … …  
    78108            g.drawImage(icon.getImage(), p.x - (width / 2), p.y - height, Main.map.mapView);
    79109        }
     110        if (noteData.getSelectedNote() != null) {
     111            StringBuilder sb = new StringBuilder("<html>");
     112            List<NoteComment> comments = noteData.getSelectedNote().getComments();
     113            String sep = "";
     114            SimpleDateFormat dayFormat = new SimpleDateFormat("MMM d, yyyy");
     115            for (NoteComment comment : comments) {
     116                String commentText = comment.getText();
     117                //closing a note creates an empty comment that we don't want to show
     118                if (commentText != null && commentText.trim().length() > 0) {
     119                    sb.append(sep);
     120                    String userName = comment.getUser().getName();
     121                    if (userName == null || userName.trim().length() == 0) {
     122                        userName = "&lt;Anonymous&gt;";
     123                    }
     124                    sb.append(userName);
     125                    sb.append(" on ");
     126                    sb.append(dayFormat.format(comment.getCommentTimestamp()));
     127                    sb.append(":<br/>");
     128                    String htmlText = XmlWriter.encode(comment.getText(), true);
     129                    htmlText = htmlText.replace("&#xA;", "<br/>"); //encode method leaves us with entity instead of \n
     130                    sb.append(htmlText);
     131                }
     132                sep = "<hr/>";
     133            }
     134            sb.append("</html>");
     135            JToolTip toolTip = new JToolTip();
     136            toolTip.setTipText(sb.toString());
     137            Point p = mv.getPoint(noteData.getSelectedNote().getLatLon());
     138
     139            g.setColor(ColorHelper.html2color(Main.pref.get("color.selected")));
     140            g.drawRect(p.x - (NoteDialog.ICON_SMALL_SIZE / 2), p.y - NoteDialog.ICON_SMALL_SIZE, NoteDialog.ICON_SMALL_SIZE - 1, NoteDialog.ICON_SMALL_SIZE - 1);
     141
     142            int tx = p.x + (NoteDialog.ICON_SMALL_SIZE / 2) + 5;
     143            int ty = p.y - NoteDialog.ICON_SMALL_SIZE - 1;
     144            g.translate(tx, ty);
     145
     146            //Carried over from the OSB plugin. Not entirely sure why it is needed
     147            //but without it, the tooltip doesn't get sized correctly
     148            for (int x = 0; x < 2; x++) {
     149                Dimension d = toolTip.getUI().getPreferredSize(toolTip);
     150                d.width = Math.min(d.width, (mv.getWidth() * 1 / 2));
     151                toolTip.setSize(d);
     152                toolTip.paint(g);
     153            }
     154            g.translate(-tx, -ty);
     155        }
    80156    }
    81157
    82158    @Override
    83159    public Icon getIcon() {
    84         return ImageProvider.get("notes", "note_open_16x16.png");
     160        return NoteDialog.ICON_OPEN_SMALL;
    85161    }
    86162
    87163    @Override
    88164    public String getToolTipText() {
    89         return notes.size() + " " + tr("Notes");
     165        return noteData.getNotes().size() + " " + tr("Notes");
    90166    }
    91167
    … …  
    111187        sb.append(tr("Total notes:"));
    112188        sb.append(" ");
    113         sb.append(notes.size());
     189        sb.append(noteData.getNotes().size());
    114190        sb.append("\n");
    115191        sb.append(tr("Changes need uploading?"));
    … …  
    128204    }
    129205
    130     /**
    131      * Returns the notes stored in this layer
    132      * @return List of Note objects
    133      */
    134     public List<Note> getNotes() {
    135         return notes;
    136     }
    137 
    138     /**
    139      * Add notes to the layer. It only adds a note if the ID is not already present
    140      * @param newNotes A list of notes to add
    141      */
    142     public void addNotes(List<Note> newNotes) {
    143         for (Note newNote : newNotes) {
    144             if (!notes.contains(newNote)) {
    145                 notes.add(newNote);
    146             }
    147         }
    148         Main.map.mapView.repaint();
    149         Main.debug("notes in layer: " + notes.size());
    150     }
     206    @Override
     207    public void mouseClicked(MouseEvent e) {
     208        if (e.getButton() != MouseEvent.BUTTON1) {
     209            return;
     210        }
     211        Point clickPoint = e.getPoint();
     212        double snapDistance = 10;
     213        double minDistance = Double.MAX_VALUE;
     214        Note closestNote = null;
     215        for (Note note : noteData.getNotes()) {
     216            Point notePoint = Main.map.mapView.getPoint(note.getLatLon());
     217            //move the note point to the center of the icon where users are most likely to click when selecting
     218            notePoint.setLocation(notePoint.getX(), notePoint.getY() - NoteDialog.ICON_SMALL_SIZE / 2);
     219            double dist = clickPoint.distanceSq(notePoint);
     220            if (minDistance > dist && clickPoint.distance(notePoint) < snapDistance ) {
     221                minDistance = dist;
     222                closestNote = note;
     223            }
     224        }
     225        noteData.setSelectedNote(closestNote);
     226    }
     227
     228    @Override
     229    public void mousePressed(MouseEvent e) { }
     230
     231    @Override
     232    public void mouseReleased(MouseEvent e) { }
     233
     234    @Override
     235    public void mouseEntered(MouseEvent e) { }
     236
     237    @Override
     238    public void mouseExited(MouseEvent e) { }
    151239}

  • trunk/src/org/openstreetmap/josm/io/NoteImporter.java

    r7538 r7608  
    5252            if (noteLayers != null && noteLayers.size() > 0) {
    5353                NoteLayer layer = noteLayers.get(0);
    54                 layer.addNotes(fileNotes);
     54                layer.getNoteData().addNotes(fileNotes);
    5555            } else {
    5656                GuiHelper.runInEDT(new Runnable() {
Note: See TracChangeset for help on using the changeset viewer.