Skip to content

Conversation

krschau
Copy link
Contributor

@krschau krschau commented Jul 11, 2022

Remove "#ifdef WinUI3" that doesn't run. Align sections of code that were in different places in the different code bases. Run all WebView2 tests down-level.

Fix issue where we were revoking m_pointerPressedRevoker twice, and revoke m_pointerReleasedRevoker instead.

@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Jul 11, 2022
@krschau
Copy link
Contributor Author

krschau commented Jul 11, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ranjeshj ranjeshj added this to the WinUI 2.8 milestone Jul 11, 2022
}

void WebView2::UpdateDefaultBackgroundColor()
void WebView2::UpdateDefaultVisualBackgroundColor()
Copy link
Contributor

Choose a reason for hiding this comment

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

UpdateDefaultVisualBackgroundColor

This function is a no-op on WinUI2, since we don't set the computed background color on anything. Do you still want to keep the code?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Does SpriteVisual have a similar background property we would ever want to use? If not, I'm happy to take it out.

Copy link
Contributor

@codendone codendone left a comment

Choose a reason for hiding this comment

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

:shipit:

@krschau
Copy link
Contributor Author

krschau commented Jul 11, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kmahone kmahone merged commit 0ce9be8 into main Jul 11, 2022
@kmahone kmahone deleted the user/krschau/wv-sync branch July 11, 2022 21:28
@ojhad ojhad added area-WebView team-Rendering Issue for the Rendering team and removed needs-triage Issue needs to be triaged by the area owners labels Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-WebView team-Rendering Issue for the Rendering team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants