action listener fixed