TableView.edit() is not working in JavaFX 2.1

Question

Hi All,
Yesterday i have updated my javafx installation from JavaFX 2.0.2 to JavaFX 2.1 b07
After that I am facing issues with TableView.edit() method. It stops working and not making the cell as editable.
In the below example , select the first cell, and on click of 'tab' , the next cell is changed to editing state..and goes on for all the cells. If it reaches the end column, the first cell of the next row is selected.
You can find the sscce example below. The same code is working well in 2.0.2 build.
import javafx.application.Application;
import javafx.beans.property.SimpleStringProperty;
import javafx.beans.value.ChangeListener;
import javafx.beans.value.ObservableValue;
import javafx.collections.FXCollections;
import javafx.collections.ObservableList;
import javafx.event.EventHandler;
import javafx.scene.Node;
import javafx.scene.Scene;
import javafx.scene.control.Label;
import javafx.scene.control.TableCell;
import javafx.scene.control.TableColumn;
import javafx.scene.control.TableView;
import javafx.scene.control.TextField;
import javafx.scene.control.cell.PropertyValueFactory;
import javafx.scene.input.KeyCode;
import javafx.scene.input.KeyEvent;
import javafx.scene.input.MouseEvent;
import javafx.scene.layout.StackPane;
import javafx.stage.Stage;
import javafx.util.Callback;
import com.sun.javafx.scene.control.skin.TableRowSkin;
public class StartEditDemo extends Application {
    public static void main(String[] args) {
        Application.launch(args);
    public void start(Stage primaryStage) {
        primaryStage.setTitle("Start Edit Test");
        StackPane root = new StackPane();
        Scene scene = new Scene(root, 600, 150);
        primaryStage.setScene(scene);
        primaryStage.show();
        configureTable(root);
     private void configureTable(StackPane root) {
          final ObservableList<MyDomain> data = FXCollections.observableArrayList(
                     new MyDomain("Apple","This is a fruit","Red"),
                     new MyDomain("PineApple","This is also a fruit","Yellow"),
                     new MyDomain("Potato","This is a vegetable","Brown")
          TableView<MyDomain> table = new TableView<MyDomain>();
          TableColumn<MyDomain,String> titleColumn = new TableColumn<MyDomain,String>("Name");
          titleColumn.setPrefWidth(150);
          titleColumn.setCellValueFactory(new PropertyValueFactory<MyDomain,String>("name"));
          titleColumn.setCellFactory(getEditableCallBack(1));
          TableColumn<MyDomain,String> descCol = new TableColumn<MyDomain,String>("Description");
          descCol.setPrefWidth(150);
          descCol.setCellValueFactory(new PropertyValueFactory<MyDomain,String>("description"));
          descCol.setCellFactory(getEditableCallBack(2));
          TableColumn<MyDomain,String> colorCol = new TableColumn<MyDomain,String>("Color");
          colorCol.setPrefWidth(150);
          colorCol.setCellValueFactory(new PropertyValueFactory<MyDomain,String>("color"));
          colorCol.setCellFactory(getEditableCallBack(0));
          table.getColumns().addAll(titleColumn,descCol,colorCol);
          table.setItems(data);
          root.getChildren().add(table);
     private Callback<TableColumn<MyDomain, String>, TableCell<MyDomain, String>> getEditableCallBack(final int nextColumn) {
          /* Cell factory call back object */
          return new Callback<TableColumn<MyDomain, String>, TableCell<MyDomain, String>>() {
               @Override
               public TableCell<MyDomain, String> call(TableColumn<MyDomain, String> p) {
                    return new EditableCell<MyDomain>(nextColumn);
     public class EditableCell<T> extends TableCell<T, String> {
          private TextField textBox;
          private Label label;
          private int nextCol;
          public EditableCell(int nextcolumn) {
               this.label = new Label();
               this.nextCol = nextcolumn;
               /* Enabling to get the edit cell on single mouse click over the cell. */
               this.setOnMouseClicked(new EventHandler<MouseEvent>() {
                    @Override
                    public void handle(MouseEvent event) {
                         startEdit();
          @Override
          public void startEdit() {
               super.startEdit();
               if (isEmpty()) {
                    return;
               if (textBox == null) {
                    createTextBox();
               } else {
                    textBox.setText(getItem());
               setGraphic(textBox);
               textBox.requestFocus();
               textBox.selectAll();
               setEditable(true);
          @Override
          public void updateItem(String item, boolean empty) {
               super.updateItem(item, empty);
               if (!isEmpty()) {
                    if (textBox != null) {
                         textBox.setText(item);
                    label.setText(item);
                    setGraphic(label);
          private void createTextBox() {
               textBox = new TextField(getItem());
               /* On focus of the textbox, selecting the row.*/
               textBox.focusedProperty().addListener(new ChangeListener<Boolean>() {
                    @Override
                    public void changed(ObservableValue<? extends Boolean> paramObservableValue,Boolean paramT1, Boolean paramT2) {
                         if(paramT2){
                              EditableCell.this.getTableView().getSelectionModel().select(EditableCell.this.getIndex());
               textBox.setOnKeyReleased(new EventHandler<KeyEvent>() {
                    @Override
                    public void handle(KeyEvent t) {
                         if (t.getCode() == KeyCode.ENTER) {
                              callAction();
                         } else if (t.getCode() == KeyCode.ESCAPE) {
                              cancelEdit();
               textBox.setOnKeyPressed(new EventHandler<KeyEvent>() {
                    @Override
                    public void handle(KeyEvent t) {
                         if (t.getCode() == KeyCode.TAB){
                              if(nextCol!=0){
                                   // Making the next as editable and focused.
                                   TableColumn nextColumn = getTableView().getColumns().get(nextCol);
                                   getTableView().edit(getTableRow().getIndex(), nextColumn);
                                   getTableView().getFocusModel().focus(getTableRow().getIndex(), nextColumn);
                              }else{
                                   // Making the next row as editable.
                                   if(getTableRow().getIndex()<getTableView().getItems().size()){
                                        TableColumn nextColumn = getTableView().getColumns().get(0);
                                        getTableView().edit(getTableRow().getIndex()+1, nextColumn);
                                        getTableView().getFocusModel().focus(getTableRow().getIndex()+1, nextColumn);
          @Override
          public void cancelEdit() {
               super.cancelEdit();
               textBox = null;
               setGraphic(label);
          @Override
          public void commitEdit(String t) {
               super.commitEdit(t);
               setItem(t);
               textBox = null;
               label.setText(t);
               setGraphic(label);
          @SuppressWarnings("rawtypes")
          public ObservableList<Node> getAllCellsOfSameRow(TableCell cell) {
               TableRowSkin rowSkin = (TableRowSkin) cell.getParent();
               return rowSkin.getChildren();
          public void callAction() {
               TableRowSkin<T> rowSkin = (TableRowSkin<T>) this.getParent();
               ObservableList<Node> cells = rowSkin.getChildren();
               for (Node node : cells) {
                    EditableCell<T> cell = (EditableCell<T>) node;
                    cell.commitEdit(cell.getTextBox().getText());
          public TextField getTextBox() {
               return this.textBox;
          public Label getLabel() {
               return this.label;
     public class MyDomain{
          private SimpleStringProperty name = new SimpleStringProperty();
          private SimpleStringProperty description = new SimpleStringProperty();
          private SimpleStringProperty color = new SimpleStringProperty();
          public MyDomain(String name, String desc, String color){
               this.name.set(name);
               this.description.set(desc);
               this.color.set(color);
          public String getDescription() {
             return description.get();
          public SimpleStringProperty descriptionProperty(){
              return description;
         public String getName() {
             return name.get();
         public SimpleStringProperty nameProperty(){
              return name;
         public String getColor() {
             return color.get();
         public SimpleStringProperty colorProperty(){
              return color;
}Regards,
Sai Pradeep Dandem.

Answer 1

As jsmith notes, the best place to report bugs is directly into the Jira tracker at http://javafx-jira.kenai.com. If you put it in the 'runtime' area against the 'controls' component, it'll end up on my plate.
Thanks!
Jonathan