It may be outside of the scope of this PR, but I wonder if it would be better/possible to use the application_helper auto_tab_index to set the tabindex.  That way we do not have to renumber the indexes when a new field is added to the view.  (auto_tab_index was initially created for this purpose, but it may not work in all cases.)