Skip to content

Conversation

@bikerduweb
Copy link
Contributor

The wrapper options must be retrieved from css_options to work for every control and it must be merged (instead of reverse_merged) because empty form_group_options values prevent a reverse_merge to work (and if we specify things in wrapper, they should always be considered as more important than default options).

Fix #153

The wrapper options must be retrieved from css_options to work for every control and it must be merged (instead of reverse_merged) because empty form_group_options values prevent a reverse_merge to work (and if we specify things in wrapper, they should always be considered as more important than default options).

Fix #153
@andrewbredow
Copy link
Contributor

I'd like to second this issue! It's currently not possible to do things like set text and class for a label. I think the intention of this code is for the user-supplied options to supersede the defaults. Currently, the logic is reversed.

potenza added a commit that referenced this pull request Jan 28, 2015
@potenza potenza merged commit 4923808 into bootstrap-ruby:master Jan 28, 2015
@bikerduweb bikerduweb deleted the patch-1 branch January 28, 2015 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

wrapper class not working

3 participants