config.show_placeholders = false fix #142

Merged
merged 1 commit into from Feb 8, 2014

Conversation

Projects
None yet
2 participants
Contributor

pedropaG commented Feb 8, 2014

config.show_placeholders only should hide the placeholder. Nor the input field too.
Condition now only evaluates :placeholder option.

config.show_placeholders = false fix
config.show_placeholders only should hide the placeholder. Nor the input field too.
Condition now only evaluates :placeholder option.

ugisozols added a commit that referenced this pull request Feb 8, 2014

Merge pull request #142 from pedropaG/patch-2
config.show_placeholders = false fix

@ugisozols ugisozols merged commit ddbc983 into refinery:master Feb 8, 2014

1 check passed

default The Travis CI build passed
Details
Owner

ugisozols commented Feb 8, 2014

Thanks @pedropaG!

Contributor

pedropaG commented Feb 10, 2014

no problem.
FYI: I think there is the same bug on 2-1-stable.

Owner

ugisozols commented Feb 10, 2014

Would you be keen to send a PR for that branch too? Thanks.

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