Skip to content

Support a Gaussian blur effect field that sometimes shows up. - #399

Merged
simeoncran merged 3 commits into
masterfrom
BlurParseFix
Dec 3, 2020
Merged

simeoncran merged 3 commits into
masterfrom
BlurParseFix

Conversation

@simeoncran

Copy link
Copy Markdown
Contributor

And polish the yaml serializer a bit.

@simeoncran
simeoncran requested a review from a team as a code owner December 2, 2020 19:09
Comment thread source/YamlData/YamlScalar.cs Outdated
@@ -28,7 +29,7 @@ public static implicit operator YamlScalar(string value)
var escapedValue = value;
if (escapedValue is null)

@simeoncran simeoncran Dec 2, 2020 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

value is not nullable. Why is escapedValue nullable? #Closed

break;

case 3:
// Optional parameter. Parse it is an animatable because it has the

@derobinsMSFT derobinsMSFT Dec 2, 2020 •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is [](start = 56, length = 2)

"as" #Closed

case 3:
// Optional parameter. Parse it is an animatable because it has the
// animatable format, however it would be very strange if the value
// was actually animated so convert it to non-animated bool;

@derobinsMSFT derobinsMSFT Dec 2, 2020 •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

; [](start = 83, length = 1)

Nit: period. #Closed

@derobinsMSFT derobinsMSFT left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@simeoncran
simeoncran merged commit a564e87 into master Dec 3, 2020
@delete-merged-branch
delete-merged-branch Bot deleted the BlurParseFix branch December 3, 2020 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants