Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Compact semantic_form_for class array #929

Merged
merged 2 commits into from Jul 4, 2013

Conversation

Projects
None yet
3 participants
Contributor

oldbonsai commented Mar 28, 2013

If the FormHelper.default_form_class is set to nil then we get extra spaces in the class in the output HTML. This will compact the classes array and knock those pesky nils right out.

Compact semantic_form_for class array
If the FormHelper.default_form_class is set to nil then we get extra spaces in the class in the output HTML. This will compact the classes array and knock those pesky nils right out.
Collaborator

sobrinho commented Mar 28, 2013

@oldbonsai can you add a spec covering that?

justinfrench added a commit that referenced this pull request Jul 4, 2013

Merge pull request #929 from oldbonsai/patch-1
Compact semantic_form_for class array

@justinfrench justinfrench merged commit b87ec6e into justinfrench:master Jul 4, 2013

1 check passed

default The Travis build passed
Details

@oldbonsai oldbonsai deleted the unknown repository branch Jul 8, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment