Skip to content

Fix warning in PropertyRM.uss - #1810

Merged
sebastienlagarde merged 1 commit into
7.x.x/releasefrom
vfx/fix/uss
Sep 10, 2020
Merged

Fix warning in PropertyRM.uss#1810
sebastienlagarde merged 1 commit into
7.x.x/releasefrom
vfx/fix/uss

Conversation

@iTris666

@iTris666 iTris666 commented Sep 10, 2020

Copy link
Copy Markdown
Contributor

Purpose of this PR

Fix warning in PropertyRM.uss


Testing status

Manual Tests: What did you do?

  • Opened test project + Run graphic tests locally
  • Built a player
  • Checked new UI names with UX convention
  • Tested UI multi-edition + Undo/Redo + Prefab overrides + Alignment in Preset
  • C# and shader warnings (supress shader cache to see them)
  • Checked new resources path for the reloader (in developer mode, you have a button at end of resources that check the paths)
  • Other:

Automated Tests: What did you setup? (Add a screenshot or the reference image of the test please)

Yamato: (Select your branch):
https://yamato.prd.cds.internal.unity3d.com/jobs/902-Graphics

Any test projects to go with this to help reviewers?


Comments to reviewers

Notes for the reviewers you have assigned.

@iTris666
iTris666 requested a review from a team as a code owner September 10, 2020 07:36
@iTris666
iTris666 changed the base branch from master to 7.x.x/release September 10, 2020 07:37
@github-actions github-actions Bot added the vfx label Sep 10, 2020
@github-actions

Copy link
Copy Markdown

It appears that you made a non-draft PR!
Please convert your PR to draft (button on the right side of the page)
and cancel any jobs that started on Yamato.
See the PR template for more information.
Thank you!

@iTris666 iTris666 changed the title Vfx/fix/uss Fix warning in PropertyRM.uss Sep 10, 2020
@sebastienlagarde
sebastienlagarde removed the request for review from a team September 10, 2020 07:42
@sebastienlagarde
sebastienlagarde merged commit 3a681e9 into 7.x.x/release Sep 10, 2020
@sebastienlagarde
sebastienlagarde deleted the vfx/fix/uss branch September 10, 2020 07:42
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.

2 participants