Gregory Casamento

Results 197 comments of Gregory Casamento
trafficstars

> @gcasa, you are also invited to review the PR, if you'd like to :) Naturally, of course, I prefer to leave it up to @rfm as he has more...

@ksuther Hey Kent, I'm @gcasa the lead developer of the project. GNUstep, of course, predates clang-format, thus we have mainly used manually formatting over the years. I personally use the...

@fredkiefer This is not yet ready for review. I will convert it when it is. Thanks.

Interesting indeed... ``` Found 20 methods on '_NSControllerTreeProxy' '_NSControllerTreeProxy' has method named 'count' of encoding 'Q16@0:8' '_NSControllerTreeProxy' has method named 'indexPath' of encoding '@16@0:8' '_NSControllerTreeProxy' has method named 'parentNode' of...

The test for this is here... https://github.com/gcasa/NSTreeController_test

Here is NSOutlineView using a dataSource... ![outline_normal](https://github.com/gnustep/libs-gui/assets/27943/a803a702-c3ba-4b69-a7c9-ee5b1bd46f1a) And here it is using a binding... ![outline_tree_controller](https://github.com/gnustep/libs-gui/assets/27943/141703b5-12eb-4061-88da-30a5e7893693) This demonstrates that the new code does not affect outline view functionality in other applications.

I am marking this as ready for review and proceeding to optimize and clean up the code in preparation for merge.

Of course, Gorm works as well... ![gorm_of_course](https://github.com/gnustep/libs-gui/assets/27943/b7bd178d-dc84-40fe-b200-a78dfc91ad0b)

@fredkiefer These changes are mainly about the binding between NSTreeController and NSOutlineView. More changes might be forthcoming, but I will do some of the other methods as separate PR. Please...

With the last change, it is now possible to update values from the outline view. Please review if possible as I feel it is now ready for merge. LMK if...