the structure of this view is confusing - to understand it future readers of the code are going to have to stop and think, look for context, etc. can it be refactored to not need conditionals or magic numbers? are we missing a presenter?