fix bug onClick