Opened 4 years ago
Closed 4 years ago
#2731 closed defect (fixed)
Entered values in Einstein dialog are ignored
| Reported by: | stoecker | Owned by: | team |
|---|---|---|---|
| Priority: | major | Component: | Core |
| Version: | Keywords: | ||
| Cc: | landwirt@… |
Description
When value in the einstein dialog are entered, they are ignored. Previously these have been ignored only when return has not been pressed. Now they are ignored always when not using the buttons.
Attachments (0)
Change History (5)
comment:1 Changed 4 years ago by stoecker
comment:2 Changed 4 years ago by Landwirt
- Cc landwirt@… added
comment:3 Changed 4 years ago by subSet
Suggested change: Table cells not editable directly. There are no listeners on the cells themselves, so we don't want to leave the impression that you can change them directly. Instead Double clicking on a field still brings up the dialog box to edit it's value, as does clicking the edit button. This behavior would be consistent with other programs such as Firefox.
simple one line fix:
Code Change:
$ svn diff
Index: src/org/openstreetmap/josm/gui/preferences/AdvancedPreference.java
===================================================================
--- src/org/openstreetmap/josm/gui/preferences/AdvancedPreference.java (revision 1685)
+++ src/org/openstreetmap/josm/gui/preferences/AdvancedPreference.java (working copy)
@@ -68,7 +68,7 @@
model = new DefaultTableModel(new String[]{tr("Key"), tr("Value")},0) {
@Override public boolean isCellEditable(int row, int column) {
- return column != 0;
+ return false;
}
};
DefaultTableCellRenderer renderer = new DefaultTableCellRenderer(){
Please someone let me know what the code review procedure is here, as I don't think I have permission yet to check in code.
comment:4 Changed 4 years ago by stoecker
I would find it much better to reenable the edit feature instead of dropping it.
comment:5 Changed 4 years ago by stoecker
- Resolution set to fixed
- Status changed from new to closed
Fixed in r1729.



Ticket #2736 has been marked as a duplicate of this ticket.