Revision: 4508
http://sourceforge.net/p/jump-pilot/code/4508
Author: michaudm
Date: 2015-06-23 22:14:40 +0000 (Tue, 23 Jun 2015)
Log Message:
-----------
AttributeTablePanel : simplify code, avoid useless selection clearing and avoid
a recursive loop [could not reproduce a case where features can be deleted from
the editing mode]
Modified Paths:
--------------
core/trunk/ChangeLog
core/trunk/src/com/vividsolutions/jump/workbench/ui/AttributeTablePanel.java
Modified: core/trunk/ChangeLog
===================================================================
--- core/trunk/ChangeLog 2015-06-23 21:55:53 UTC (rev 4507)
+++ core/trunk/ChangeLog 2015-06-23 22:14:40 UTC (rev 4508)
@@ -1,5 +1,10 @@
# for display continuity sake please use 2 spaces instead of tabs
+2015-06-24 mmichaud <m.michael.mich...@orange.fr>
+ * AttributeTablePanel : simplify code, avoid useless selection clearing and
avoid a
+ recursive loop [could not reproduce a case where features can be deleted
from the
+ editing mode]
+
2015-06-23 bertazza
* improved legend handling for rasters (via RasterStyler):
* the legend is now integrated in the layer tree of the TOC
Modified:
core/trunk/src/com/vividsolutions/jump/workbench/ui/AttributeTablePanel.java
===================================================================
---
core/trunk/src/com/vividsolutions/jump/workbench/ui/AttributeTablePanel.java
2015-06-23 21:55:53 UTC (rev 4507)
+++
core/trunk/src/com/vividsolutions/jump/workbench/ui/AttributeTablePanel.java
2015-06-23 22:14:40 UTC (rev 4508)
@@ -735,6 +735,7 @@
// pressing a key starts the cell editing mode, but it did not
// clear the selection and kept delete feature action possible,
// which was quite dangerous. Now, the selection is cleared
+ /*
table.addKeyListener(new java.awt.event.KeyListener(){
public void keyPressed(java.awt.event.KeyEvent e) {
if (!layer.isEditable()) return;
@@ -754,6 +755,7 @@
public void keyTyped(java.awt.event.KeyEvent e) {
}
});
+ */
// Just after it has been created, AttributeTablePanel listen to
the layerView selection
// to be able to reflect the view selection into the table
workbenchContext.getLayerViewPanel().addListener(layerViewPanelListener);
@@ -1089,7 +1091,7 @@
* @param panel
*/
public void selectionReplaced(AttributeTablePanel panel) {
- if (table.isEditing()) table.clearSelection();
+
int[] selectedRows = table.getSelectedRows();
// After selectedRows have been memorized, clear the layer selection,
// other wise OpenJUMP will add the selectedRows to the already
selected features
------------------------------------------------------------------------------
Monitor 25 network devices or servers for free with OpManager!
OpManager is web-based network management software that monitors
network devices and physical & virtual servers, alerts via email & sms
for fault. Monitor 25 devices for free with no restriction. Download now
http://ad.doubleclick.net/ddm/clk/292181274;119417398;o
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
