melistik / vaadin-grid-util

simplify the use of vaadin's grid and add features
https://vaadin.com/directory/component/gridutil
MIT License
32 stars 21 forks source link

both edit / delete combination of button fire onDelete (issue 11# persist) #13

Closed lipodido closed 9 years ago

lipodido commented 9 years ago

with the 1.0.7 the issue still there with EDIT <--> DELETE combination

.....new EditDeleteButtonValueRenderer(new EditDeleteButtonClickListener() {......

Both get to onDelete

regards

melistik commented 9 years ago

sorry but on my end it's working:

// change line 113 in DemoUI and i get 2 differnt notifications
grid.getColumn("id")
        .setRenderer(new EditDeleteButtonValueRenderer(new EditDeleteButtonValueRenderer.EditDeleteButtonClickListener() {

            @Override
            public void onEdit(final RendererClickEvent event) {
                Notification.show(event.getItemId()
                        .toString() + " want's to get edited", Type.HUMANIZED_MESSAGE);
            }

            @Override
            public void onDelete(final RendererClickEvent event) {
                Notification.show(event.getItemId()
                        .toString() + " want's to get deleted", Type.WARNING_MESSAGE);
            }
        }))
        .setWidth(160);
lipodido commented 9 years ago

ok after recompiling the widgetset everything goes fine , sorry it was my fault :(