Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added hooks for extending the Reflection API for custom data types and Elements #66

Closed
wants to merge 8 commits into from

Conversation

davidjade
Copy link

I added some hooks for extending the Reflection API. It's a simple design that I used for when I had custom UI views that I wanted to bind to custom data value types. This allowed me to continue to use the reflection API data binding yet still get custom UI in a simple way.

It might be interesting to use these hooks for the basis of all Element building and value fetching to make the system more flexible (but I did not do that here).

@migueldeicaza
Copy link
Owner

Thanks for the patch, but could you also extend the sample to show its use?

@davidjade
Copy link
Author

sure, it might take me a few days to get it though. I'll probably just tack on an example to the existing reflection sample.

@davidjade
Copy link
Author

Had some time to work on this. I added a demo using a simple custom view to bind to a custom data type using the reflection extension. It shows all the basics of how this is intended to be used.

@davidjade
Copy link
Author

I've made a few changes to the reflection extension API to make it more flexible but I also found a bug where the Detail Text is not always cleared when RootElement cells get reused. This bug seems to exists regardless of any of my changes.

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.

2 participants