MRCollective / ChameleonForms

Shape-shifting your forms experience in ASP.NET Core MVC
MIT License
254 stars 56 forks source link

.TabIndex() on IFieldConfiguration #75

Closed robdmoore closed 10 years ago

robdmoore commented 10 years ago

Add a TabIndex(int index) method in IFieldConfiguration that adds tabindex="{index}" to the field.

See the Placeholder method for an example of a similar method and find usages to see where it's been tested etc.

Trello Issue: https://trello.com/c/LUTMVmdD/155-tabindex-on-ifieldconfiguration

chandu commented 10 years ago

@robdmoore I think TabIndex, AutoFocus etc (including Placeholder) are better implemented as extension methods, since these methods just need adding the entries to Attributes dictionary? This way the IFieldConfiguration interface need not be modified/bloated? Your thoughts?

robdmoore commented 10 years ago

Hi @Chandu!

Awesome idea - I completely agree.