Skip to content

EMSUSD-3311 - Add remove opinion + undo#4651

Merged
seando-adsk merged 3 commits into
devfrom
barbalt/dev/EMSUSD-3311-remove-opinion
Jun 18, 2026
Merged

EMSUSD-3311 - Add remove opinion + undo#4651
seando-adsk merged 3 commits into
devfrom
barbalt/dev/EMSUSD-3311-remove-opinion

Conversation

@barbalt

@barbalt barbalt commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator
image

@barbalt barbalt self-assigned this Jun 16, 2026
@barbalt barbalt force-pushed the barbalt/dev/EMSUSD-3311-remove-opinion branch from 2090249 to 56095e6 Compare June 16, 2026 20:14
@barbalt barbalt removed their assignment Jun 16, 2026
@barbalt barbalt force-pushed the barbalt/dev/EMSUSD-3311-remove-opinion branch from 56095e6 to 9f01a62 Compare June 17, 2026 12:55
@barbalt barbalt force-pushed the barbalt/dev/EMSUSD-3311-remove-opinion branch from 9f01a62 to d0b22a5 Compare June 17, 2026 13:00
@barbalt barbalt self-assigned this Jun 17, 2026
@barbalt barbalt changed the title EMSUSD-3311 - Add undo to remove opinion EMSUSD-3311 - Add remove opinion + undo Jun 17, 2026
@barbalt barbalt requested a review from deboisj June 17, 2026 13:03
@seando-adsk seando-adsk added ready-for-merge Development process is finished, PR is ready for merge shared_components Shared Components (Max / Maya) labels Jun 18, 2026
@seando-adsk

Copy link
Copy Markdown
Collaborator

Talis ran an internal preflight build using this github branch and his internal branch which updates the Composition View component. That build was all green so marking this PR as ready for merge.

@seando-adsk seando-adsk merged commit e31ce5b into dev Jun 18, 2026
10 checks passed
@seando-adsk seando-adsk deleted the barbalt/dev/EMSUSD-3311-remove-opinion branch June 18, 2026 13:07
deboisj pushed a commit that referenced this pull request Jun 26, 2026
…-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
4AWa68v0v1dc4ii
 ieHece6ACMGNWY52RnOOxj1sNO3OFGVuVpjR6HvUSxWsWwErW+9jESZ1WI+Bok6D
 +xB9K6wkANvUs1nQDnGpvUzzaVu1ngl4FLgyd74Xzqh0kK505WUcG89/0XRmJf5Y
 zm//WtVMXJYAUZbMRJPG5pW+bIzt7rpqgXagXVUUA3+gQWOdtGMF1WGRD0Vequ2E
 Uh2S6bgLx1z3Sh73iFHmfXhy80kI3ZJwfLNOPdFvGo0pFL3XFfV84fypVXVD+7fA
 tNbR5ap6xtPm3gRWm5Pj8dNhY8hA9D0FwsbV84ZaMhpJU8SrNxWZ9UJpniwLp+mQ
 3oEIxKntXRJL/ulVyBZNYEgB1h00k6QOXZiqqnZ1BkVDks/vAF7Fdbj50gfNCXAD
 IS+AcbR97VfqLcK1bXLqx+C4uThoQjEQFvgKZ1udk3aCu9Q/LPG3eEDkmQsYsucK
 rZIVqRLalDb8KKBqAsdtIwd4FdsztyA13wVWrHAx5VaV3ayPufoXPukXuZukRRMA
 KA1MutisuUm2xGEu5PIC
 =6Nuz
 -----END PGP SIGNATURE-----
 

Merge pull request #4649 from dj-mcg/pr/fix-vtValueToUfeValue-ifdef-guard

Fix undefined vtValueToUfeValue when UFE_SCENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
red-By: Claude Opus 4.8 <noreply@anthropic.com>
ly@anthropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
…editor_review

kQACksGu6ycD7dV7/lRp4Guvvj
 KVCsT9UwEthfdkPDMfa7G3i+9GSacK733THUcJppZK+4DM2m8PbSiP0pj/dSaElf
 iaICFZ7H2KJ6tTashv3bQPy6Z7mn4slht5L+66l+D4rWifT8QB1CXj2z2cgZ+2NX
 oc1LH6nKO6HKmq8crOB63Yb8O81jLOWCJoYt40nrlxy1SWqIJpMNnV2mj+1xAnUI
 9n1h3qJb4+Jauu4zH+JPds/dewJ+3sLwKSF5q8BdLb8XmHckHG2ajvXmpZAJg6Rk
 vtkQDip8Viem0NpWOsVRVuQxOeqLs1UXhZt5e3bZ5txHR5L5PiHeb0rrphZ7b57w
 50zBFPlKnP/+noIbMBtPBSUkw9B0lQFNVvxjamQic2Zga23J5+dcJ1Qf9J1tVbf+
 hDOY1r3vxFU1PnDPvJjTgsA0ATH+P33Z1wQ4lYNaENo2iT+nbccq/fOusSNzsBdl
 iK2auWyxHX4P21ZpsZlICRgSkUn68Ncn0dJIUA6grewUAGxoBGkGHQ17FTTxWtXe
 XaZ3WpikFeNcRS89DayGpcjnj9UzSco2L0ZrudCLh/ltNDrxlk4tpQQrMep+0aI+
 B/oIBncZS8y4IZAKdDm5pqJtGaYulqwYHAXL41QyAv/QEg+sA20KLSRKVIaQz7CK
 kq12Y3YgiDpN7fbCmJx/
 =2NPd
 -----END PGP SIGNATURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
: Claude Opus 4.8 <noreply@anthropic.com>
ly@anthropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
876036 -0400

Merge remote-tracking branch 'upstream/dev' into deboisj/unify_layer_editor_review
kQACksGu6ycD7dV7/lRp4Guvvj
 KVCsT9UwEthfdkPDMfa7G3i+9GSacK733THUcJppZK+4DM2m8PbSiP0pj/dSaElf
 iaICFZ7H2KJ6tTashv3bQPy6Z7mn4slht5L+66l+D4rWifT8QB1CXj2z2cgZ+2NX
 oc1LH6nKO6HKmq8crOB63Yb8O81jLOWCJoYt40nrlxy1SWqIJpMNnV2mj+1xAnUI
 9n1h3qJb4+Jauu4zH+JPds/dewJ+3sLwKSF5q8BdLb8XmHckHG2ajvXmpZAJg6Rk
 vtkQDip8Viem0NpWOsVRVuQxOeqLs1UXhZt5e3bZ5txHR5L5PiHeb0rrphZ7b57w
 50zBFPlKnP/+noIbMBtPBSUkw9B0lQFNVvxjamQic2Zga23J5+dcJ1Qf9J1tVbf+
 hDOY1r3vxFU1PnDPvJjTgsA0ATH+P33Z1wQ4lYNaENo2iT+nbccq/fOusSNzsBdl
 iK2auWyxHX4P21ZpsZlICRgSkUn68Ncn0dJIUA6grewUAGxoBGkGHQ17FTTxWtXe
 XaZ3WpikFeNcRS89DayGpcjnj9UzSco2L0ZrudCLh/ltNDrxlk4tpQQrMep+0aI+
 B/oIBncZS8y4IZAKdDm5pqJtGaYulqwYHAXL41QyAv/QEg+sA20KLSRKVIaQz7CK
 kq12Y3YgiDpN7fbCmJx/
 =2NPd
 -----END PGP SIGNATURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
om shared LE
876036 -0400

Merge remote-tracking branch 'upstream/dev' into deboisj/unify_layer_editor_review
kQACksGu6ycD7dV7/lRp4Guvvj
 KVCsT9UwEthfdkPDMfa7G3i+9GSacK733THUcJppZK+4DM2m8PbSiP0pj/dSaElf
 iaICFZ7H2KJ6tTashv3bQPy6Z7mn4slht5L+66l+D4rWifT8QB1CXj2z2cgZ+2NX
 oc1LH6nKO6HKmq8crOB63Yb8O81jLOWCJoYt40nrlxy1SWqIJpMNnV2mj+1xAnUI
 9n1h3qJb4+Jauu4zH+JPds/dewJ+3sLwKSF5q8BdLb8XmHckHG2ajvXmpZAJg6Rk
 vtkQDip8Viem0NpWOsVRVuQxOeqLs1UXhZt5e3bZ5txHR5L5PiHeb0rrphZ7b57w
 50zBFPlKnP/+noIbMBtPBSUkw9B0lQFNVvxjamQic2Zga23J5+dcJ1Qf9J1tVbf+
 hDOY1r3vxFU1PnDPvJjTgsA0ATH+P33Z1wQ4lYNaENo2iT+nbccq/fOusSNzsBdl
 iK2auWyxHX4P21ZpsZlICRgSkUn68Ncn0dJIUA6grewUAGxoBGkGHQ17FTTxWtXe
 XaZ3WpikFeNcRS89DayGpcjnj9UzSco2L0ZrudCLh/ltNDrxlk4tpQQrMep+0aI+
 B/oIBncZS8y4IZAKdDm5pqJtGaYulqwYHAXL41QyAv/QEg+sA20KLSRKVIaQz7CK
 kq12Y3YgiDpN7fbCmJx/
 =2NPd
 -----END PGP SIGNATURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
sts
om shared LE
876036 -0400

Merge remote-tracking branch 'upstream/dev' into deboisj/unify_layer_editor_review
kQACksGu6ycD7dV7/lRp4Guvvj
 KVCsT9UwEthfdkPDMfa7G3i+9GSacK733THUcJppZK+4DM2m8PbSiP0pj/dSaElf
 iaICFZ7H2KJ6tTashv3bQPy6Z7mn4slht5L+66l+D4rWifT8QB1CXj2z2cgZ+2NX
 oc1LH6nKO6HKmq8crOB63Yb8O81jLOWCJoYt40nrlxy1SWqIJpMNnV2mj+1xAnUI
 9n1h3qJb4+Jauu4zH+JPds/dewJ+3sLwKSF5q8BdLb8XmHckHG2ajvXmpZAJg6Rk
 vtkQDip8Viem0NpWOsVRVuQxOeqLs1UXhZt5e3bZ5txHR5L5PiHeb0rrphZ7b57w
 50zBFPlKnP/+noIbMBtPBSUkw9B0lQFNVvxjamQic2Zga23J5+dcJ1Qf9J1tVbf+
 hDOY1r3vxFU1PnDPvJjTgsA0ATH+P33Z1wQ4lYNaENo2iT+nbccq/fOusSNzsBdl
 iK2auWyxHX4P21ZpsZlICRgSkUn68Ncn0dJIUA6grewUAGxoBGkGHQ17FTTxWtXe
 XaZ3WpikFeNcRS89DayGpcjnj9UzSco2L0ZrudCLh/ltNDrxlk4tpQQrMep+0aI+
 B/oIBncZS8y4IZAKdDm5pqJtGaYulqwYHAXL41QyAv/QEg+sA20KLSRKVIaQz7CK
 kq12Y3YgiDpN7fbCmJx/
 =2NPd
 -----END PGP SIGNATURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
…rage

Apply the LOW_VALUE_TEST_REVIEW decisions (drop/strengthen/merge crash-only and
tautological tests), then restore the coverage those removals dropped:

- Restore testLayerTreeViewMouseLogic.cpp and testStageSelectorWidgetLogic.cpp
  (strengthened) and re-add them to CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
++ tests: remove low-value tests, recover coverage

Apply the LOW_VALUE_TEST_REVIEW decisions (drop/strengthen/merge crash-only and
tautological tests), then restore the coverage those removals dropped:

- Restore testLayerTreeViewMouseLogic.cpp and testStageSelectorWidgetLogic.cpp
  (strengthened) and re-add them to CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
les.
++ tests: remove low-value tests, recover coverage

Apply the LOW_VALUE_TEST_REVIEW decisions (drop/strengthen/merge crash-only and
tautological tests), then restore the coverage those removals dropped:

- Restore testLayerTreeViewMouseLogic.cpp and testStageSelectorWidgetLogic.cpp
  (strengthened) and re-add them to CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
ve low-value tests, recover coverage

Apply the LOW_VALUE_TEST_REVIEW decisions (drop/strengthen/merge crash-only and
tautological tests), then restore the coverage those removals dropped:

- Restore testLayerTreeViewMouseLogic.cpp and testStageSelectorWidgetLogic.cpp
  (strengthened) and re-add them to CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
nification review pass
ve low-value tests, recover coverage

Apply the LOW_VALUE_TEST_REVIEW decisions (drop/strengthen/merge crash-only and
tautological tests), then restore the coverage those removals dropped:

- Restore testLayerTreeViewMouseLogic.cpp and testStageSelectorWidgetLogic.cpp
  (strengthened) and re-add them to CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
ass
ve low-value tests, recover coverage

Apply the LOW_VALUE_TEST_REVIEW decisions (drop/strengthen/merge crash-only and
tautological tests), then restore the coverage those removals dropped:

- Restore testLayerTreeViewMouseLogic.cpp and testStageSelectorWidgetLogic.cpp
  (strengthened) and re-add them to CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
 tests. Review pass
ass
ve low-value tests, recover coverage

Apply the LOW_VALUE_TEST_REVIEW decisions (drop/strengthen/merge crash-only and
tautological tests), then restore the coverage those removals dropped:

- Restore testLayerTreeViewMouseLogic.cpp and testStageSelectorWidgetLogic.cpp
  (strengthened) and re-add them to CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hropic.com>
deboisj added a commit that referenced this pull request Jun 26, 2026
…p QA_TEST_REVIEW scratch doc

T_REVIEW decisions (drop/strengthen/merge crash-only and
tautological tests), then restore the coverage those removals dropped:

- Restore testLayerTreeViewMouseLogic.cpp and testStageSelectorWidgetLogic.cpp
  (strengthened) and re-add them to CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hropic.com>
deboisj pushed a commit that referenced this pull request Jun 26, 2026
…xportChaser` or `ImportChaser`.

- Refactors the shared plugin discovery/loading logic used by jobContext and shadingMode registries.
- Reuses that logic for import and export chasers, triggering chaser plugin loading just before chasers are created.
o CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
eply@anthropic.com>
hropic.com>
deboisj pushed a commit that referenced this pull request Jun 26, 2026
ster an `ExportChaser` or `ImportChaser`.

- Refactors the shared plugin discovery/loading logic used by jobContext and shadingMode registries.
- Reuses that logic for import and export chasers, triggering chaser plugin loading just before chasers are created.
o CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
eply@anthropic.com>
hropic.com>
deboisj pushed a commit that referenced this pull request Jun 26, 2026
ster an `ExportChaser` or `ImportChaser`.

- Refactors the shared plugin discovery/loading logic used by jobContext and shadingMode registries.
- Reuses that logic for import and export chasers, triggering chaser plugin loading just before chasers are created.
o CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
eply@anthropic.com>
hropic.com>
deboisj pushed a commit that referenced this pull request Jun 26, 2026
tChaser` or `ImportChaser`.

- Refactors the shared plugin discovery/loading logic used by jobContext and shadingMode registries.
- Reuses that logic for import and export chasers, triggering chaser plugin loading just before chasers are created.
o CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
eply@anthropic.com>
hropic.com>
deboisj pushed a commit that referenced this pull request Jun 26, 2026
…ugins

Automatic loading of  `ExportChaser` and `ImportChaser` pluginsis no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
ic for import and export chasers, triggering chaser plugin loading just before chasers are created.
o CMakeLists.
- Add strengthened LayerTreeView tests (mute/lock button-push, key Delete/R,
  add-parent, expand-children) and LayerEditorWidget tests (splitter/lazy-update/
  save-stage button, GeneratedIconButton paint states).
- Add SaveLayersDialog all-as-relative tests and an unguarded EF toggle-button
  test covering the runtime-gated edit-forwarding setup.
- Export GeneratedIconButton (LayerEditorAPI) so its paint paths are testable.

UsdLayerEditorNewTests: 438 tests, 437 pass, 1 expected skip. Lib coverage ~70%.
TURE-----
 

Merge pull request #4651 from Autodesk/barbalt/dev/EMSUSD-3311-remove-opinion

EMSUSD-3311 - Add remove opinion + undoENEITEM_HAS_METADATA is no…both unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
eply@anthropic.com>
hropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge Development process is finished, PR is ready for merge shared_components Shared Components (Max / Maya)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants