Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@slunity
Copy link
Contributor

@slunity slunity commented Oct 12, 2020

(fogbugz 1257050)

Checklist for PR maker

  • Have you added a backport label (if needed)? For example, the need-backport-* label. After you backport the PR, the label changes to backported-*.
  • Have you updated the changelog? Each package has a CHANGELOG.md file.

Purpose of this PR

Why is this PR needed, what hard problem is it solving/fixing?
Fix https://fogbugz.unity3d.com/f/cases/1257050/


Testing status

Describe what manual/automated tests were performed for this PR
Create any ShaderGraph using HDRP Lit target, check graph settings in graph inspector, switch material modes.


Comments to reviewers

I added backport for 9.x since the fogbugz case mentions it.

…urface option properties are only displayed respectively for SSS and SpecularColor material modes.

(fogbugz 1257050)
@sebastienlagarde sebastienlagarde merged commit 1b5bc83 into HDRP/staging Oct 12, 2020
@sebastienlagarde sebastienlagarde deleted the hdrp-fix-litsg-surfaceoptions-display branch October 12, 2020 13:18
@ghost ghost removed the needs-backport-9.x label Oct 27, 2020
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.