Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Canvas SizeChanged is not triggered when it is inside another Canvas #2900

Closed
cnbluefire opened this issue Jul 14, 2020 · 3 comments
Closed
Labels
area-Layouts bug Something isn't working needs-winui-3 Indicates that feature can only be done in WinUI 3.0 or beyond. (needs winui 3) no-issue-activity wct

Comments

@cnbluefire
Copy link

Describe the bug
When modifying the width and height of the Canvas, If it's directly in another Canvas, ActualWidth and ActualHeight will change but SizeChanged is not triggered.

Steps to reproduce the bug
like this: https://github.com/cnbluefire/CanvasSizeChangedTest

Expected behavior
When a control's width and height change, it should trigger SizeChanged

Version Info

NuGet package version:

Windows 10 version Saw the problem?
Insider Build (xxxxx)
May 2020 Update (19041)
November 2019 Update (18363)
May 2019 Update (18362)
October 2018 Update (17763) Yes
April 2018 Update (17134)
Fall Creators Update (16299)
Creators Update (15063)
Device form factor Saw the problem?
Desktop Yes
Xbox
Surface Hub
IoT

Additional context

@msft-github-bot msft-github-bot added the needs-triage Issue needs to be triaged by the area owners label Jul 14, 2020
@chrisglein chrisglein added area-Layouts bug Something isn't working needs-winui-3 Indicates that feature can only be done in WinUI 3.0 or beyond. (needs winui 3) and removed needs-triage Issue needs to be triaged by the area owners labels Jul 17, 2020
@michael-hawker
Copy link
Collaborator

michael-hawker commented Sep 8, 2021

I was also noticing that if an element within the Canvas is moved (via Canvas.Top or Canvas.Left) that element's LayoutUpdated event isn't fired. So I don't think there's a way for the element to detect that it's moved either... I think that'd be a separate bug though. maybe? Definitely seems related in some ways. @ranjeshj know if that's the expected behavior or not?

@michael-hawker
Copy link
Collaborator

Related to #674

@github-actions
Copy link

This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Layouts bug Something isn't working needs-winui-3 Indicates that feature can only be done in WinUI 3.0 or beyond. (needs winui 3) no-issue-activity wct
Projects
None yet
Development

No branches or pull requests

5 participants