Skip to content

Conversation

fredericv-unity3d
Copy link
Contributor

@fredericv-unity3d fredericv-unity3d commented Apr 28, 2020

Purpose of this PR

Fix case 1136601.

Issue: when using Perforce, there is no way to checkout a profile for edition from the Volume Inspector.

I drawed the VCS toolbar used for the asset inside the editor, so we have a fast access to the VCS features of the volume profile.

TODO:

  • UX validation
  • Documentation update
  • Update changelog
  • Should we backport?

Previous UI:
image

New UI:
WTmeMp4JaY

Rationale

  • Why using the full toolbar? Easier for technical reason and to keep UX consistency. We are drawing the inspector of another asset inside the volume inspector.

Testing status

Manual Tests:

  • Tested UI in different situation:
    • Perforce configured/not configured
    • Asset checked out/not checked out
    • Reverting asset
    • Committing asset

Comments to reviewers

@fredericv-unity3d fredericv-unity3d changed the title Hdrp/fix perforce volume Add a checkout button on Volume Profile field when using Perforce Apr 28, 2020
@fredericv-unity3d fredericv-unity3d force-pushed the HDRP/fix_perforce_volume branch from 928bcf3 to f479593 Compare April 30, 2020 09:56
@fredericv-unity3d fredericv-unity3d changed the title Add a checkout button on Volume Profile field when using Perforce Draw the VCS toolbar when Perforce is activated in the Volume Editor Apr 30, 2020
@sebastienlagarde sebastienlagarde deleted the HDRP/fix_perforce_volume branch September 1, 2021 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant