refactored to about controller