should this be a class instead of inline style, or is it better inline?