This is totally pedantic, but I was initially read this code I couldn't understand exactly why `value` was being set like this.  Should it be changed from

    value = options[:value]
    field_config = document_show_fields(document)[field]
    value ||= get_field_values document, field, field_config, options

    render_field_value value

to something like

    field_config = document_show_fields(document)[field]
    value = options[:value] || get_field_values document, field, field_config, options

    render_field_value value

If we do decide to change it I believe the same pattern exists above in `render_index_field_value`.