Modify

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

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

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.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.