Modify

Opened 15 years ago

Closed 15 years ago

Last modified 7 years ago

#2731 closed defect (fixed)

Entered values in Einstein dialog are ignored

Reported by: stoecker Owned by: team
Priority: major Milestone:
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 by stoecker, 15 years ago

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

comment:2 by Landwirt, 15 years ago

Cc: landwirt@… added

comment:3 by subSet, 15 years ago

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 by stoecker, 15 years ago

I would find it much better to reenable the edit feature instead of dropping it.

comment:5 by stoecker, 15 years ago

Resolution: fixed
Status: newclosed

Fixed in r1729.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.