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

Registration mechanism for custom column in SceneHierarchy #820

Open
andrewkaufman opened this issue Apr 30, 2014 · 4 comments
Open

Registration mechanism for custom column in SceneHierarchy #820

andrewkaufman opened this issue Apr 30, 2014 · 4 comments
Labels
ui Issues related to the UI/UX in Gaffer

Comments

@andrewkaufman
Copy link
Contributor

This would allow external additions of display information specific for the user. Relates to #48 and #80.

@andrewkaufman andrewkaufman added this to the IERendering 7 milestone Apr 30, 2014
@ldmoser ldmoser modified the milestones: SceneInspector and SceneHierarchy, Tag Support Apr 30, 2014
@johnhaddon
Copy link
Member

It would be handy to have examples of the sorts of things people will want to display here. I'll start us off with a couple :

  • Visibility
  • Shader assignment (maybe colour coded?)

@johnhaddon
Copy link
Member

As we add features to the SceneHierarchy, we're going to need to keep a close eye on performance. The SceneHierarchy is one of our worst performing UI elements, because it actually deals with every item in the scene (when expanded), and does so using Python. It's the one place we access the whole scene without using C++ to do it. At some point I think we'll need to reimplement our Gaffer.Path class in C++, and then reimplement the PathListingWidget _PathModel() in C++. That will have some interesting problems, like how do we write C++ Qt stuff, and get it into Python for both PyQt and PySide. And it'll also bring with it more fun build permutations, because we'll need to build and link with the right Qt for each host application, instead of just worrying about it at the Python level...

@yutapple
Copy link

@markwendell was asking for this too :)

@yutapple
Copy link

aannnd would be great if it grays out when visibility attribute was set to hidden :)

@andrewkaufman andrewkaufman added ui Issues related to the UI/UX in Gaffer and removed ui labels Jun 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui Issues related to the UI/UX in Gaffer
Projects
Status: On the Radar
Development

No branches or pull requests

4 participants