-
Notifications
You must be signed in to change notification settings - Fork 208
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
Draw vertical connector in SceneHierarchy to make parent child relationships clearer. #81
Comments
I've spent all morning faffing with this, and come to the conclusion that it's not possible to do it the way Ben drew using stylesheets. Qt seems to want it to always be along these lines : The difference being that the "show the children" arrow doesn't connect to the children it shows, instead it connects to the next sibling. I actually find this pretty unintuitive compared to Ben's example, so I'm not sure what to do. It looks like we might be able to do it the way we want if we took over the drawing entirely, but that would either have speed implications or mean C++ implementation which to date we've avoided for Qt - it would mean many more headaches when hosting in 3rd party apps. Any thoughts? Would you rather have rubbish unintuitive lines or no lines? Or wait till we inevitably have to do some C++ to speed up the scene hierarchy and then do it properly? |
The other thing we could do in a future version is to highlight the parent items of the current selection. Qt is horribly slow when dealing with sparse selections (which are hard to make in the Scene Hierarchy but easy to make in the viewer) so at some point I think we'll have to do our own selection drawing anyway. It would be natural to use a PathMatcher for that, and we could very easily use a DescendantMatch query to draw highlighted parent items. |
I'd vote to wait. We're not using SceneHierarchy in production yet, and we're going to overhaul it for speed at some future point anyways. Not sure why this one slipped into a 3d viewer milestone in the first place... |
Specifically helpful for large scenes with long lists of children.
Something along these lines might work.
![hierarchyConnections](https://camo.githubusercontent.com/d570b4fb5aef82566ec676c1fb350c4d2cb94b1f75269199151adcaa27fd89e9/68747470733a2f2f662e636c6f75642e6769746875622e636f6d2f6173736574732f333937373632382f3332393937382f32393237313737652d396263322d313165322d393163302d6130396237303863323833362e706e67)
The text was updated successfully, but these errors were encountered: