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.

Sign up for GitHub

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

Jump to bottom

Hot Reload not refresh the UI if apply changes related with sizing (Padding, Margin, etc) #4611

Closed
jsuarezruiz opened this issue Feb 10, 2022 · 3 comments
Closed

Hot Reload not refresh the UI if apply changes related with sizing (Padding, Margin, etc) #4611

jsuarezruiz opened this issue Feb 10, 2022 · 3 comments
Labels
area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter area-tooling XAML & C# Hot Reload, XAML Editor, Live Visual Tree, Live Preview, Debugging high It doesn't work at all, crashes or has a big impact. partner/ui-tools .NET UI Tools platform/android 🤖 platform/windows 🪟 s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
Triage
Milestone
6.0.300-rc.3

Comments

@jsuarezruiz
Copy link
Contributor

Description

Hot Reload not refresh the UI if apply changes related with sizing (Padding, Margin, etc).
hr-notworking

Steps to Reproduce

  1. Download https://github.com/jsuarezruiz/netmaui-chat-app-challenge
  2. Navigate to the chat message ItemTemplate and modify any property that affects the position of an element (Paddding, Margin, etc).

Version with bug

Preview 12 (current)

Last version that worked well

Unknown/Other

Affected platforms

Android, Windows

Affected platform versions

Windows SDK 10.0.17134.0, Android 11

Did you find any workaround?

No response

Relevant log output

No response
@jsuarezruiz jsuarezruiz added t/bug Something isn't working area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter area-tooling XAML & C# Hot Reload, XAML Editor, Live Visual Tree, Live Preview, Debugging high It doesn't work at all, crashes or has a big impact. labels Feb 10, 2022
@jsuarezruiz jsuarezruiz added this to New in Triage via automation Feb 10, 2022
@jsuarezruiz jsuarezruiz moved this from New to Ready For Work in Triage Feb 10, 2022
@jsuarezruiz jsuarezruiz mentioned this issue Feb 10, 2022
Open
@drasticactions
Copy link
Contributor

@jsuarezruiz Try poking the values from C# code behind (Like making a button that will change the padding and margin on those fields). I'm 99% sure it won't work there either.

@drasticactions drasticactions added the partner/ui-tools .NET UI Tools label Feb 10, 2022
@wenwen60
Copy link

wenwen60 commented Feb 25, 2022 •
edited
Loading

Reproduce on Android 11 using VS2022 17.2.0 Preview 2.0 [32215.517.main] with above project.

@wenwen60 wenwen60 added the s/verified Verified / Reproducible Issue ready for Engineering Triage label Feb 25, 2022
@Redth Redth added this to the 6.0.300-rc.2 milestone Mar 22, 2022
@kristinx0211 kristinx0211 mentioned this issue Apr 14, 2022
Closed
@Redth Redth modified the milestones: 6.0.300-rc.2, 6.0.300-rc.3 Apr 20, 2022
@davidortinau
Copy link
Contributor

Cannot reproduce this with main (rc2). Looks to be resolved.

Triage automation moved this from Ready For Work to Done Apr 22, 2022
@ghost ghost locked as resolved and limited conversation to collaborators May 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter area-tooling XAML & C# Hot Reload, XAML Editor, Live Visual Tree, Live Preview, Debugging high It doesn't work at all, crashes or has a big impact. partner/ui-tools .NET UI Tools platform/android 🤖 platform/windows 🪟 s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
No open projects
Triage
Done
Milestone
6.0.300-rc.3
Development

No branches or pull requests
6 participants
@davidortinau @Redth @drasticactions @jsuarezruiz @samhouts @wenwen60