search

MRCollective / ChameleonForms

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

Add possibility to replace EnumListHandler with custom implementation. #... #102

Closed zabulus closed 9 years ago

zabulus commented 10 years ago

https://github.com/MRCollective/ChameleonForms/issues/101

zabulus commented 10 years ago

It looks good if you want to customize an enum field in a model. But if you want to customize it in all enum properties in all models, need to modify code in Form.cs to utilize a custom IFieldGenerator. But I think it is better to add some extensibility to the FieldGeneratorHandlersRouter. Thoughts aloud. All main problem in the extensibility of the project it is generics. You can't add FieldGenerator.DefaultFactory for all Model/Types. You can do it only from main enter point - Form.

robdmoore commented 10 years ago

FYI, I haven't forgotten this. I'm pondering the best way to achieve it keeping in mind my ideas for the future direction of the library. cc @MattDavies

zabulus commented 10 years ago

sure, np. I've fixed my issue, but already had started the implementation of extension.