-
Notifications
You must be signed in to change notification settings - Fork 202
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
EMSUSD-1541 - Drag and drop one or more scene objects from the outliner into the include or exclude list #4030
Conversation
|
||
if not prim or not prim.IsValid(): | ||
raise ValueError("Value must be a float or an int")("Error: The dragged object is not in the same stage as the collection. Ensure that objects belong to the same stage before adding them") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if there's a better way than ValueError in this case, trying to avoid using Maya or UFE specific methods...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In other points of the code there's a
raise RuntimeError("Needs to be implemented in derived class")
So I think the ValueError should be fine here
@@ -60,6 +61,11 @@ def __init__(self, items: Sequence[str] = None, parent=None): | |||
|
|||
self.selectionModel().selectionChanged.connect(lambda: self.selectedItemsChanged.emit()) | |||
|
|||
self.placeholder_label = QLabel("Drag objects here or click “+” to add", self) | |||
self.placeholder_label.setAlignment(Qt.AlignmentFlag.AlignCenter) | |||
self.placeholder_label.setStyleSheet("color: gray; font-size: 18px;") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a bit worried about this specific color and pixel size here to work on max as well with different dpi monitors
I'll test those internally first |
No description provided.