fixed style
updated controller
updated to work with values rather than name