Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[WPF] Fix issue setting initial ScrollView properties (VerticalScrollBarVisibility, etc) #11918

Merged
merged 3 commits into from
Sep 17, 2020

Conversation

jsuarezruiz
Copy link
Contributor

Description of Change

Fix issue setting initial ScrollView properties (VerticalScrollBarVisibility, etc) on WPF

Issues Resolved

API Changes

None

Platforms Affected

  • WPF

Behavioral/Visual Changes

None

Before/After Screenshots

fix11709

Testing Procedure

Launch Core Gallery and navigate to the issue 11709. If even being able to scroll in the ScrollView you cannot see the vertical scrollbar, the test has passed.

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

[assembly: Xamarin.Forms.Dependency(typeof(Deserializer))]

[assembly: AssemblyVersion("2.0.0.0")]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This changes are related more with #11328 just to be able to launch Core Gallery and test everything.

@samhouts samhouts added retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. and removed retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. labels Aug 25, 2020
@samhouts samhouts changed the base branch from 4.8.0 to 5.0.0 August 26, 2020 21:28
@samhouts samhouts added the approved Has two approvals, no pending reviews, and no changes requested label Sep 8, 2020
@@ -7,7 +7,7 @@
<OutputType>WinExe</OutputType>
<UseWPF>true</UseWPF>
<AppendTargetFrameworkToOutputPath>true</AppendTargetFrameworkToOutputPath>
<GenerateAssemblyInfo>false</GenerateAssemblyInfo>
<GenerateAssemblyInfo>true</GenerateAssemblyInfo>
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we need this change.

@rmarinho rmarinho merged commit f35ae07 into 5.0.0 Sep 17, 2020
@rmarinho rmarinho deleted the fix-11709 branch September 17, 2020 10:55
@samhouts samhouts added this to the 5.0.0 milestone Sep 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/scrollview approved Has two approvals, no pending reviews, and no changes requested ControlGallery p/WPF t/bug 🐛
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] [WPF] ScrollView - ScrollBarVisibility not respected
4 participants