`position_key` made more sense to me re: what the top bit is finding, so I went with it. Don't feel strongly. 

Sent from my iPhone

On Jul 16, 2013, at 21:30, Carlos Antonio da Silva <notifications@github.com> wrote:

> In lib/simple_form/inputs/date_time_input.rb:
> 
> >  
> > -        position = ActionView::Helpers::DateTimeSelector::POSITION[position]
> > -        "#{attribute_name}_#{position}i"
> > +        position_value = ActionView::Helpers::DateTimeSelector::POSITION[position_key]
> > +        "#{attribute_name}_#{position_value}i"
> Any specific reason to change the variable names and not sticking with position only? It seems to just make the diff harder to follow since the real change is the to_sym call.
> 
> Thansk.
> 
> —
> Reply to this email directly or view it on GitHub.