Open matsbryntse opened 3 months ago
Could you please consider to add in MenuItem also:
item.trigger('action', itemEvent);
So we can have one handler definition in namedItems instead of 2, so it can be re-used for both buttons and menu items:
namedItems: { removeRow: { text: 'Remove row', onItem: 'ctrl.onRemoveRowClick', onAction: 'ctrl.onRemoveRowClick', }, },
=>
namedItems: { removeRow: { text: 'Remove row', onAction: 'ctrl.onRemoveRowClick', }, },
Originally posted by @vadimpopa in https://github.com/bryntum/support/issues/8266#issuecomment-1925414302
Could you please consider to add in MenuItem also:
item.trigger('action', itemEvent);
So we can have one handler definition in namedItems instead of 2, so it can be re-used for both buttons and menu items:
=>
Originally posted by @vadimpopa in https://github.com/bryntum/support/issues/8266#issuecomment-1925414302