Fix issue with refresh of view based NSTableView#403
Conversation
|
I really would prefer if you could do formatting changes as a separate commit. This can be in the same PR. That way it would be simpler to look through the remaining changes. |
Okay. Understood. I will do so going forward. |
|
I am going to see if there are other solutions to this problem before marking it ready. |
|
@fredkiefer This is ready for review, I don't think I can simplify it more. |
fredkiefer
left a comment
There was a problem hiding this comment.
As I wrote in my mail. I am not happy with this PR in its current state. But if you really want to merge it, go ahead
I haven't been able to find one, but I think you're right. I am going to give it another week to try to find a better solution. |
…DiffableDataSource_fixes
The issue that I am trying to fix is that when the scrollbar is used the number of rows in the NSTableView shows ONE row and not all. I am not sure why this is happening.