Skip to content

Fix/canvas gallery #676

Open
Drout wants to merge 2 commits intomicrosoft:mainfrom
Drout:fix/canvas-gallery-dotnet8
Open

Fix/canvas gallery #676
Drout wants to merge 2 commits intomicrosoft:mainfrom
Drout:fix/canvas-gallery-dotnet8

Conversation

@Drout
Copy link
Copy Markdown

@Drout Drout commented May 4, 2026

Summary

This PR contains two related changes:

  1. Fixes canvas gallery item typing so gallery item access resolves child controls correctly in published apps.
  2. Retargets PowerAppsTestEngineWrapper to .NET 8 to eliminate warnings and align with the current dependency set.

What changed

Canvas gallery item support

  • Added RunJavascriptInFrameAsync<T> to the test infra so the canvas provider can evaluate JavaScript directly inside the published app iframe.
  • Updated the published app JS object-model builder to reconstruct gallery AllItems / Items and Selected / Default property types from gallery child controls.
  • Added a provider-side fallback that patches gallery child typing when the runtime still reports empty collection/object shapes.

This enables expressions that depend on gallery item child controls, such as indexing into gallery items and then accessing nested control properties.

Wrapper target framework update

  • Changed PowerAppsTestEngineWrapper from netstandard2.0 to net8.0.
  • Removed package settings that were only needed for the previous target/warning profile.

Validation

  • Built src/PowerAppsTestEngine.sln successfully in Debug configuration with 0 warnings and 0 errors.

@Drout Drout requested a review from a team as a code owner May 4, 2026 13:01
@Drout
Copy link
Copy Markdown
Author

Drout commented May 4, 2026

@microsoft-github-policy-service agree

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.

1 participant