crisu83 / yiistrap

Twitter Bootstrap for the Yii PHP framework.
http://www.getyiistrap.com
BSD 3-Clause "New" or "Revised" License
286 stars 176 forks source link

Reverted attributes placeholders in activeTextInputField #287

Closed magefad closed 9 years ago

magefad commented 9 years ago

Previously used the old version bootstrap (YiiBooster, DrMabuse bootstrap 3). And all of them are displayed by default placeholders. After the upgrade project to use yiistrap all forms have empty placeholders.

May be solution is - add the setting to enable or disable this feature, for backward compatibility...

crisu83 commented 9 years ago

This is all good but it would be even better if you'd also add a test for it as well.

crisu83 commented 9 years ago

I don't think this is a good idea, the default behavior in Bootstrap itself is not to have placeholders, but labels for inputs.