Skip to content

Fix bug with hamburgermenu and right align #2

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

Closed
deltakosh opened this issue Jun 17, 2016 · 1 comment
Closed

Fix bug with hamburgermenu and right align #2

deltakosh opened this issue Jun 17, 2016 · 1 comment
Labels
bug 🐛 An unexpected issue that highlights incorrect behavior wontfix ❌ Discontinued Issues or PR
Milestone

Comments

@deltakosh
Copy link
Contributor

Go to the sample app and onthe hamburger test page select right for PanePlacement

@deltakosh deltakosh added the bug 🐛 An unexpected issue that highlights incorrect behavior label Jun 17, 2016
@deltakosh deltakosh added this to the v0.5 milestone Jun 17, 2016
@deltakosh
Copy link
Contributor Author

This is related to the datatemplate only

@deltakosh deltakosh added the wontfix ❌ Discontinued Issues or PR label Jun 29, 2016
deltakosh pushed a commit that referenced this issue Sep 2, 2016
deltakosh pushed a commit that referenced this issue Oct 26, 2016
hermitdave pushed a commit that referenced this issue Jan 11, 2017
Pull from main MS repo
ScottIsAFool pushed a commit that referenced this issue Apr 29, 2017
nmetulev pushed a commit that referenced this issue May 3, 2017
nmetulev pushed a commit that referenced this issue Oct 26, 2017
IbraheemOsama pushed a commit that referenced this issue Nov 15, 2017
WilliamABradley pushed a commit that referenced this issue Jan 23, 2018
azchohfi pushed a commit that referenced this issue Apr 16, 2018
nmetulev pushed a commit that referenced this issue May 2, 2018
nmetulev pushed a commit that referenced this issue May 6, 2018
Merge from original to update fork with latest
nmetulev pushed a commit that referenced this issue May 17, 2018
azchohfi pushed a commit that referenced this issue May 22, 2018
ryalanms added a commit to ryalanms/WindowsCommunityToolkit that referenced this issue Aug 25, 2018
GraniteStateHacker added a commit that referenced this issue Aug 28, 2018
* Folded the latest work into the CTK.  This includes wrappers for MediaPlayerElement, InkCanvas, and InkToolbar.  There's also a new Samples app called TestSample (for the time being).  In order to accomplish this, the projects in the Win32 sln needed to be updated to .NET 4.7.2 and SDK 17686.

* Added WPF MapControl wrapper.

* Initial WPF wraps for SwapChainPanel & WebView.

* refactors setter implementation for consistency.  Fixes some missing event handlers in WebView (Lite).

* Minimally working demos of InkToolbar with InkToolBarCustomToolButton as a wrapped child control.  Not quite happy with this implementation, but wanted to bounce it off the team.

* Sets the appropriate type on the Wrapper attached property

* 95% code-generated wrappers (minor post-process touchups only)

* Fixes the binding issue on MediaPlayer property for one-way binding.

* Refactored events to use a dynamic wrapper for event args.

* Updates events to pattern that works, requiring event args wrappers. (thus adding 25 event args wrappers) and an additional control wrapper.

* WIP:  Refactoring WebView for fallback behavior.

* Committing this so I can test another branch.

* Restoring samples for Ink, Mediaplayer, et al.

* Checkpointing

* InkPresenter's pretty well fleshed out.

* Fixes compile issue.

* Pulled back on winmd ref changes.

* Map control is in the sample project now, and doesn't die, but doesn't show a map yet, either.

* looking into binding issues with the map control, but not convinced that's the issue.

* Added SwapChainPanel to sample

* Fixes references so that Debug & Release both build

* Attempts to apply dpi fixes to test sample project.  Also fixes implementation selection condition for WebViewCompatible.

* Updates required after merge.

* Removed SwapChainPanel, MapControl, and hopefully extraneous InkPresenter code.

* Revert "Removed SwapChainPanel, MapControl, and hopefully extraneous InkPresenter code."

This reverts commit 5f977c3.

* Ran UpdateHeaders.bat

* Experimentation with full control set.

* Restored to .NET 4.6.2

* Removed SecurityCriticalAttribute from class, not needed.

* Fixed a naming issue that surfaced in review.

* Got rid of the ".Compatible" namespace for the WebViewCompatible control.

* renamed Microsoft.Windows.Interop to Microsoft.Toolkit.Win32.UI.Interop. Convertet project to enable pack

* Regenerated wrappers including basic redirect pointers for documentation.  Included a few more wrappers and enums to simplify and reduce need for WinMD.

* split up Microsoft.Toolkit.Win32.UI.Controls in WPF and WinForms binaries sharing common code

* Fix references

- Add missing project references
- Use Windows.winmd set by Directory.Build
- Convert GAC references to NuGet packages

* updated headers

* Worked in the latest WindowsXamlHost / WindowsXamlHostBase (with a local extension to enable needed functionality).

* More properly integrates Interop.WPF code, extending WindowsXamlHostBase to sort out differences.

* Fixing some sample project breakages in an earlier brancrh to prove out the branch.

* Splitting WindowsXamlHost off to WPF & WinForms.  This is the WPF path.

* Pulls in Interop.WinForms.

* Moves wpf shared code in by link.

* Set IsFullscreenButtonVisibile = false in MediaPlayerElement since it's not supported.

* Moved the sample app to a more appropriate namespace / assembly name in prep for importing WinForms sample.  Also hammered out WebViewCompatible.

* Getting the WinForms Sample app in.

* Ran UpdateHeaders.bat

* Updated to use latest SDK (17713) via Directory.Build.props file.

* Making sure the samples uses Directory.Build.props target platform version.

* Got the WebViewCompatible partially operable.

* Almost there, checkpointing for a pull from rimuri/xaml-islands_more-fixes.

* StyleCop fixes for Microsoft.Toolkit.Win32.UI.Interop.WinForms

Some fixes still required for documentation and usage in some cases.

* StyleCop fixes for Microsoft.Toolkit.Win32.UI.Interop.WPF

* Optimize references

* Fixing a comment in directory.build.props.

* Fix commas in WindowsXamlHostBase.cs

* Fix StyleCop issues with DependencyObjectCollection.cs

* designer tweak

* fixes binding on the source property.

* Updated initialization of WebView in WebViewCompatibilityAdapter.

* Working out stylecop errors.

* Refactored to get rid of Windows dependency creeping out of WindowsXamlHostBase.

* fixed up release build configuration

* reverted tpm for all projects

* Set docking state before EndInit

Asynchronous initialization occurs after calling EndInit. To prevent multiple paints when the control is resized during initialization, move the dock style prior to EndInit to ensure the target HWND RECT.

* Correct order of members

* Implement IDisposable

WebView is disposable. Making WebViewCompatibilityAdapter disposable and dispose of WebView

* DRY out WinRT type detection

WebViewCompatible duplicated WinMD metadata detection

* Correct order of members

* Correct issues with merge

* Add missing file headers

* Remove redundant `partial` keyword

* Remove default type of int for enum

`int` is already governing type for an enum

* Remove `global::` prefix for WinRT types

The ability to access a member in the global namespace is useful when a member might be hidden by another entity of the same name.

Example
```
class Foo
{
  public class System { }

  const int Console = 0;

  static void Main()
  {
    Console.WriteLine("Bar");
  }
}
```
The `Console.WriteLine` throws an error because it accesses Foo.Console, which is a constant. Using `System.Console` does no better, since it accesses `Foo.System` class, which does not have a static `WriteLine` method. The only way to get it to compile is to use `global::System.Console.WriteLine("Bar")`; however, it is not required in these instances.

* ThreadStaticAttribute does not do anything for instance members

* Move WinRT interop files to proper namespace

Follows convention used by WebView

* Clean up InkCanvas

- Remove `this`
- Use `inheritdoc` for documentation generation
- Remove unused using statement

* Clean up InkToolbar.cs

- Remove unused using
- Remove `this`

* Clean up InkToolbarCustomPenButton.cs

- Remove `this`

* Clean up InkToolbarCustomToolButton.cs

- Remove unused usings
- Remove `this`
- Remove `global::` alias

* Clean up InkToolbarIsStencilButtonCheckChangedEventArgs.cs

- Remove unused usings
- Simplify names
- Fix property `StencilKind` to use interop type instead of WinRT

* Clean up InkToolbarMenuButton.cs

- Remove unused using
- Remove `this`
- Remove cast to int on `MenuKind`. Enum is already int, so no need to cast then cast to enum.

* Clean up InkToolbarPenButton.cs

- Remove unused using
- Remove `this`

* Clean up InkToolbarStencilButton.cs

- Remove unused usings
- Remove `this`
- Remove `global::` in most places
- Convert `Ruler` to expression bodied member

* Clean up InkToolbarToggleButton.cs

- Remove unused usings
- Remove `this`
- Convert `ToggleKind` to expression body, remove unnecessary cast to int
- Remove `global::` in most places

* Clean up InkToolbarToolButton.cs

- Remove unused usings
- Remove `this`
- Remove most instances of `global::`
- Convert `ToolKind` to expression body, remove unnecessary cast to int

* Clean up BindingBase.cs

- Remove `this`

* Clean up BindingExpression.cs

- Remove `this`

* Clean up BringIntoViewOptions.cs

- Remove `this`

* Clean up Brush.cs

- Remove `this`
- Left review comments for uncaught NRE

* Clean up BrushTransition.cs

- Remove `this`

* Clean up MediaPlayerElement.cs

- Remove unused usings
- Remove `this`

* Leave review comment in MediaSourceConverter.cs

* Remove `Support` folder

Contents moved to WinRT/Interop folder

* Clean up WindowsXamlHostBaseExt.cs

- Add inheritdoc for documentation
- Remove `this`

* Clean up IWebViewCompatible.cs

- Remove unused using
- Group members (all properties, methods, events, etc. together)

* Clean up WebBaseCompatabilityAdapter.cs

- Group members

* Provide Web adapters correct implementation for IDispose pattern

* Clean up WebBrowserCompatabilityAdapter.cs

- Implement proper IDisposable
- Add null checks prior to operations to avoid ObjectDisposeException or NullRefrenceException

* Clean up WebViewCompatabilityAdapter.cs

- Add null checks for operations to avoid unhandled NRE

* Work in progress

* Fixing a merge error.

* updated folder names and updated toolbox xml file

* Add app.config/app.manifest to WinForms sample app

Required to support high DPI

* added visualstudiotoolsmanidest.xml to nuget package

* WIP while updating from master

* Designer now allows association between ink toolbar & its canvas.

* Ink canvas / toolbar updates, dug in on mediaplayerelement.

* MediaPlayerElement oprational to the same level of functionality as WPF edition.

* Project rename fix

* Fixing a reference that broke in merge.

* Ran UpdateHeaders.bat.

* Fixed release build config.

* Adding stub MDs for control documents.

* Add link to ExternDll.cs

* Adding md documentation for wrappers.

* Remove duplicate tag from build props
Move WebBrowserUriTypeConverter to shared

* Adding missing task extensions for both WPF & WinForms.

* Addressing a PR comment about comments.

* Adding no-op Pack targets for the build.

* removed samples apps from building in release mode

* Tweaked docs.

* Fixing TargetPlatformVersion / TargetPlatformMinVersion

* WIP

* Removing code from accidental commit from winforms branch.

* Temp solution to InkToolbar issue.

* set min target version

* Resolves review issues.

* Addressing PR 2299 change requests.

* Cosmetic change to trigger another build to see if it overcomes the OOM error in the previous build.

* updated win32 project to 17733

* updated comments

* removed Shadow.cs because Windows.UI.Xaml.Media.Shadow is removed from latest insider build. Replaced WebViewControl.AddpreLoadedScript with AddInitializeScript

* Clears up event handler on dispose of WidowsXamlHost.

* Renaming packages in #2299 (#2420)

* renamed win32 namespaces

* updated vs toolbox namespaces

* Fix for local build for PR #2490 merged into #2299.

* Added missing projects to Release Build config.

* Revert "Fix for local build for PR #2490 merged into #2299."

This reverts commit d9202e0.

* Update NativeDefines

- Move to correct namespace
- Update to use existing definitions for WM

* Update Win32 code

Move Win32 code into appropriate namespace. Fix issues with `NativeMethods` style classes and add documentation about when to use which and for what.

* Update namespace

- Remove unused namespaces
- Move `using` outside of `namespace` declaration

* Merge branch 'rjmurillo/XamlIslands' into rjmurillo/XamlIslands-ControlTest

* Fix AssemblyInfos for XamlHosts

* API feedback part 1

* API updates #2

* Code docs updates

- Separate remarks into `<remarks />` tag
- Link types with `<see />` tag
- Fix some typos

* Changed the way SetContent works in WindowsXamlHostBase so that it doesn't expose Windows.winmd to the client.
rjmurillo pushed a commit that referenced this issue Aug 28, 2018
GraniteStateHacker added a commit that referenced this issue Sep 6, 2018
* Folded the latest work into the CTK.  This includes wrappers for MediaPlayerElement, InkCanvas, and InkToolbar.  There's also a new Samples app called TestSample (for the time being).  In order to accomplish this, the projects in the Win32 sln needed to be updated to .NET 4.7.2 and SDK 17686.

* Added WPF MapControl wrapper.

* Initial WPF wraps for SwapChainPanel & WebView.

* refactors setter implementation for consistency.  Fixes some missing event handlers in WebView (Lite).

* Minimally working demos of InkToolbar with InkToolBarCustomToolButton as a wrapped child control.  Not quite happy with this implementation, but wanted to bounce it off the team.

* Sets the appropriate type on the Wrapper attached property

* 95% code-generated wrappers (minor post-process touchups only)

* Fixes the binding issue on MediaPlayer property for one-way binding.

* Refactored events to use a dynamic wrapper for event args.

* Updates events to pattern that works, requiring event args wrappers. (thus adding 25 event args wrappers) and an additional control wrapper.

* WIP:  Refactoring WebView for fallback behavior.

* Committing this so I can test another branch.

* Restoring samples for Ink, Mediaplayer, et al.

* Checkpointing

* InkPresenter's pretty well fleshed out.

* Fixes compile issue.

* Pulled back on winmd ref changes.

* Map control is in the sample project now, and doesn't die, but doesn't show a map yet, either.

* looking into binding issues with the map control, but not convinced that's the issue.

* Added SwapChainPanel to sample

* Fixes references so that Debug & Release both build

* Attempts to apply dpi fixes to test sample project.  Also fixes implementation selection condition for WebViewCompatible.

* Updates required after merge.

* Removed SwapChainPanel, MapControl, and hopefully extraneous InkPresenter code.

* Revert "Removed SwapChainPanel, MapControl, and hopefully extraneous InkPresenter code."

This reverts commit 5f977c3.

* Ran UpdateHeaders.bat

* Experimentation with full control set.

* Restored to .NET 4.6.2

* Removed SecurityCriticalAttribute from class, not needed.

* Fixed a naming issue that surfaced in review.

* Got rid of the ".Compatible" namespace for the WebViewCompatible control.

* renamed Microsoft.Windows.Interop to Microsoft.Toolkit.Win32.UI.Interop. Convertet project to enable pack

* Regenerated wrappers including basic redirect pointers for documentation.  Included a few more wrappers and enums to simplify and reduce need for WinMD.

* split up Microsoft.Toolkit.Win32.UI.Controls in WPF and WinForms binaries sharing common code

* Fix references

- Add missing project references
- Use Windows.winmd set by Directory.Build
- Convert GAC references to NuGet packages

* updated headers

* Worked in the latest WindowsXamlHost / WindowsXamlHostBase (with a local extension to enable needed functionality).

* More properly integrates Interop.WPF code, extending WindowsXamlHostBase to sort out differences.

* Fixing some sample project breakages in an earlier brancrh to prove out the branch.

* Splitting WindowsXamlHost off to WPF & WinForms.  This is the WPF path.

* Pulls in Interop.WinForms.

* Moves wpf shared code in by link.

* Set IsFullscreenButtonVisibile = false in MediaPlayerElement since it's not supported.

* Moved the sample app to a more appropriate namespace / assembly name in prep for importing WinForms sample.  Also hammered out WebViewCompatible.

* Getting the WinForms Sample app in.

* Ran UpdateHeaders.bat

* Updated to use latest SDK (17713) via Directory.Build.props file.

* Making sure the samples uses Directory.Build.props target platform version.

* Got the WebViewCompatible partially operable.

* Almost there, checkpointing for a pull from rimuri/xaml-islands_more-fixes.

* StyleCop fixes for Microsoft.Toolkit.Win32.UI.Interop.WinForms

Some fixes still required for documentation and usage in some cases.

* StyleCop fixes for Microsoft.Toolkit.Win32.UI.Interop.WPF

* Optimize references

* Fixing a comment in directory.build.props.

* Fix commas in WindowsXamlHostBase.cs

* Fix StyleCop issues with DependencyObjectCollection.cs

* designer tweak

* fixes binding on the source property.

* Updated initialization of WebView in WebViewCompatibilityAdapter.

* Working out stylecop errors.

* Refactored to get rid of Windows dependency creeping out of WindowsXamlHostBase.

* fixed up release build configuration

* reverted tpm for all projects

* Set docking state before EndInit

Asynchronous initialization occurs after calling EndInit. To prevent multiple paints when the control is resized during initialization, move the dock style prior to EndInit to ensure the target HWND RECT.

* Correct order of members

* Implement IDisposable

WebView is disposable. Making WebViewCompatibilityAdapter disposable and dispose of WebView

* DRY out WinRT type detection

WebViewCompatible duplicated WinMD metadata detection

* Correct order of members

* Correct issues with merge

* Add missing file headers

* Remove redundant `partial` keyword

* Remove default type of int for enum

`int` is already governing type for an enum

* Remove `global::` prefix for WinRT types

The ability to access a member in the global namespace is useful when a member might be hidden by another entity of the same name.

Example
```
class Foo
{
  public class System { }

  const int Console = 0;

  static void Main()
  {
    Console.WriteLine("Bar");
  }
}
```
The `Console.WriteLine` throws an error because it accesses Foo.Console, which is a constant. Using `System.Console` does no better, since it accesses `Foo.System` class, which does not have a static `WriteLine` method. The only way to get it to compile is to use `global::System.Console.WriteLine("Bar")`; however, it is not required in these instances.

* ThreadStaticAttribute does not do anything for instance members

* Move WinRT interop files to proper namespace

Follows convention used by WebView

* Clean up InkCanvas

- Remove `this`
- Use `inheritdoc` for documentation generation
- Remove unused using statement

* Clean up InkToolbar.cs

- Remove unused using
- Remove `this`

* Clean up InkToolbarCustomPenButton.cs

- Remove `this`

* Clean up InkToolbarCustomToolButton.cs

- Remove unused usings
- Remove `this`
- Remove `global::` alias

* Clean up InkToolbarIsStencilButtonCheckChangedEventArgs.cs

- Remove unused usings
- Simplify names
- Fix property `StencilKind` to use interop type instead of WinRT

* Clean up InkToolbarMenuButton.cs

- Remove unused using
- Remove `this`
- Remove cast to int on `MenuKind`. Enum is already int, so no need to cast then cast to enum.

* Clean up InkToolbarPenButton.cs

- Remove unused using
- Remove `this`

* Clean up InkToolbarStencilButton.cs

- Remove unused usings
- Remove `this`
- Remove `global::` in most places
- Convert `Ruler` to expression bodied member

* Clean up InkToolbarToggleButton.cs

- Remove unused usings
- Remove `this`
- Convert `ToggleKind` to expression body, remove unnecessary cast to int
- Remove `global::` in most places

* Clean up InkToolbarToolButton.cs

- Remove unused usings
- Remove `this`
- Remove most instances of `global::`
- Convert `ToolKind` to expression body, remove unnecessary cast to int

* Clean up BindingBase.cs

- Remove `this`

* Clean up BindingExpression.cs

- Remove `this`

* Clean up BringIntoViewOptions.cs

- Remove `this`

* Clean up Brush.cs

- Remove `this`
- Left review comments for uncaught NRE

* Clean up BrushTransition.cs

- Remove `this`

* Clean up MediaPlayerElement.cs

- Remove unused usings
- Remove `this`

* Leave review comment in MediaSourceConverter.cs

* Remove `Support` folder

Contents moved to WinRT/Interop folder

* Clean up WindowsXamlHostBaseExt.cs

- Add inheritdoc for documentation
- Remove `this`

* Clean up IWebViewCompatible.cs

- Remove unused using
- Group members (all properties, methods, events, etc. together)

* Clean up WebBaseCompatabilityAdapter.cs

- Group members

* Provide Web adapters correct implementation for IDispose pattern

* Clean up WebBrowserCompatabilityAdapter.cs

- Implement proper IDisposable
- Add null checks prior to operations to avoid ObjectDisposeException or NullRefrenceException

* Clean up WebViewCompatabilityAdapter.cs

- Add null checks for operations to avoid unhandled NRE

* Work in progress

* Fixing a merge error.

* updated folder names and updated toolbox xml file

* Add app.config/app.manifest to WinForms sample app

Required to support high DPI

* added visualstudiotoolsmanidest.xml to nuget package

* WIP while updating from master

* Designer now allows association between ink toolbar & its canvas.

* Ink canvas / toolbar updates, dug in on mediaplayerelement.

* MediaPlayerElement oprational to the same level of functionality as WPF edition.

* Project rename fix

* Fixing a reference that broke in merge.

* Ran UpdateHeaders.bat.

* Fixed release build config.

* Adding stub MDs for control documents.

* Add link to ExternDll.cs

* Adding md documentation for wrappers.

* Remove duplicate tag from build props
Move WebBrowserUriTypeConverter to shared

* Adding missing task extensions for both WPF & WinForms.

* Addressing a PR comment about comments.

* Adding no-op Pack targets for the build.

* removed samples apps from building in release mode

* Tweaked docs.

* Fixing TargetPlatformVersion / TargetPlatformMinVersion

* WIP

* Removing code from accidental commit from winforms branch.

* Temp solution to InkToolbar issue.

* set min target version

* Resolves review issues.

* Addressing PR 2299 change requests.

* Cosmetic change to trigger another build to see if it overcomes the OOM error in the previous build.

* updated win32 project to 17733

* updated comments

* removed Shadow.cs because Windows.UI.Xaml.Media.Shadow is removed from latest insider build. Replaced WebViewControl.AddpreLoadedScript with AddInitializeScript

* Clears up event handler on dispose of WidowsXamlHost.

* Renaming packages in #2299 (#2420)

* renamed win32 namespaces

* updated vs toolbox namespaces

* Update NativeDefines

- Move to correct namespace
- Update to use existing definitions for WM

* Update Win32 code

Move Win32 code into appropriate namespace. Fix issues with `NativeMethods` style classes and add documentation about when to use which and for what.

* Update namespace

- Remove unused namespaces
- Move `using` outside of `namespace` declaration

* Merge branch 'rjmurillo/XamlIslands' into rjmurillo/XamlIslands-ControlTest

* Fix AssemblyInfos for XamlHosts

* API feedback part 1

* API updates #2

* Code docs updates

- Separate remarks into `<remarks />` tag
- Link types with `<see />` tag
- Fix some typos

* fixed up solution and csproj

* Add Windows Forms Focus workaround for RS5

* Revert "Revert "Introduction of XamlHost""

This reverts commit 5200188.

* Jimwilcox/xaml islands control test (#2431)

* Folded the latest work into the CTK.  This includes wrappers for MediaPlayerElement, InkCanvas, and InkToolbar.  There's also a new Samples app called TestSample (for the time being).  In order to accomplish this, the projects in the Win32 sln needed to be updated to .NET 4.7.2 and SDK 17686.

* Added WPF MapControl wrapper.

* Initial WPF wraps for SwapChainPanel & WebView.

* refactors setter implementation for consistency.  Fixes some missing event handlers in WebView (Lite).

* Minimally working demos of InkToolbar with InkToolBarCustomToolButton as a wrapped child control.  Not quite happy with this implementation, but wanted to bounce it off the team.

* Sets the appropriate type on the Wrapper attached property

* 95% code-generated wrappers (minor post-process touchups only)

* Fixes the binding issue on MediaPlayer property for one-way binding.

* Refactored events to use a dynamic wrapper for event args.

* Updates events to pattern that works, requiring event args wrappers. (thus adding 25 event args wrappers) and an additional control wrapper.

* WIP:  Refactoring WebView for fallback behavior.

* Committing this so I can test another branch.

* Restoring samples for Ink, Mediaplayer, et al.

* Checkpointing

* InkPresenter's pretty well fleshed out.

* Fixes compile issue.

* Pulled back on winmd ref changes.

* Map control is in the sample project now, and doesn't die, but doesn't show a map yet, either.

* looking into binding issues with the map control, but not convinced that's the issue.

* Added SwapChainPanel to sample

* Fixes references so that Debug & Release both build

* Attempts to apply dpi fixes to test sample project.  Also fixes implementation selection condition for WebViewCompatible.

* Updates required after merge.

* Removed SwapChainPanel, MapControl, and hopefully extraneous InkPresenter code.

* Revert "Removed SwapChainPanel, MapControl, and hopefully extraneous InkPresenter code."

This reverts commit 5f977c3.

* Ran UpdateHeaders.bat

* Experimentation with full control set.

* Restored to .NET 4.6.2

* Removed SecurityCriticalAttribute from class, not needed.

* Fixed a naming issue that surfaced in review.

* Got rid of the ".Compatible" namespace for the WebViewCompatible control.

* renamed Microsoft.Windows.Interop to Microsoft.Toolkit.Win32.UI.Interop. Convertet project to enable pack

* Regenerated wrappers including basic redirect pointers for documentation.  Included a few more wrappers and enums to simplify and reduce need for WinMD.

* split up Microsoft.Toolkit.Win32.UI.Controls in WPF and WinForms binaries sharing common code

* Fix references

- Add missing project references
- Use Windows.winmd set by Directory.Build
- Convert GAC references to NuGet packages

* updated headers

* Worked in the latest WindowsXamlHost / WindowsXamlHostBase (with a local extension to enable needed functionality).

* More properly integrates Interop.WPF code, extending WindowsXamlHostBase to sort out differences.

* Fixing some sample project breakages in an earlier brancrh to prove out the branch.

* Splitting WindowsXamlHost off to WPF & WinForms.  This is the WPF path.

* Pulls in Interop.WinForms.

* Moves wpf shared code in by link.

* Set IsFullscreenButtonVisibile = false in MediaPlayerElement since it's not supported.

* Moved the sample app to a more appropriate namespace / assembly name in prep for importing WinForms sample.  Also hammered out WebViewCompatible.

* Getting the WinForms Sample app in.

* Ran UpdateHeaders.bat

* Updated to use latest SDK (17713) via Directory.Build.props file.

* Making sure the samples uses Directory.Build.props target platform version.

* Got the WebViewCompatible partially operable.

* Almost there, checkpointing for a pull from rimuri/xaml-islands_more-fixes.

* StyleCop fixes for Microsoft.Toolkit.Win32.UI.Interop.WinForms

Some fixes still required for documentation and usage in some cases.

* StyleCop fixes for Microsoft.Toolkit.Win32.UI.Interop.WPF

* Optimize references

* Fixing a comment in directory.build.props.

* Fix commas in WindowsXamlHostBase.cs

* Fix StyleCop issues with DependencyObjectCollection.cs

* designer tweak

* fixes binding on the source property.

* Updated initialization of WebView in WebViewCompatibilityAdapter.

* Working out stylecop errors.

* Refactored to get rid of Windows dependency creeping out of WindowsXamlHostBase.

* fixed up release build configuration

* reverted tpm for all projects

* Set docking state before EndInit

Asynchronous initialization occurs after calling EndInit. To prevent multiple paints when the control is resized during initialization, move the dock style prior to EndInit to ensure the target HWND RECT.

* Correct order of members

* Implement IDisposable

WebView is disposable. Making WebViewCompatibilityAdapter disposable and dispose of WebView

* DRY out WinRT type detection

WebViewCompatible duplicated WinMD metadata detection

* Correct order of members

* Correct issues with merge

* Add missing file headers

* Remove redundant `partial` keyword

* Remove default type of int for enum

`int` is already governing type for an enum

* Remove `global::` prefix for WinRT types

The ability to access a member in the global namespace is useful when a member might be hidden by another entity of the same name.

Example
```
class Foo
{
  public class System { }

  const int Console = 0;

  static void Main()
  {
    Console.WriteLine("Bar");
  }
}
```
The `Console.WriteLine` throws an error because it accesses Foo.Console, which is a constant. Using `System.Console` does no better, since it accesses `Foo.System` class, which does not have a static `WriteLine` method. The only way to get it to compile is to use `global::System.Console.WriteLine("Bar")`; however, it is not required in these instances.

* ThreadStaticAttribute does not do anything for instance members

* Move WinRT interop files to proper namespace

Follows convention used by WebView

* Clean up InkCanvas

- Remove `this`
- Use `inheritdoc` for documentation generation
- Remove unused using statement

* Clean up InkToolbar.cs

- Remove unused using
- Remove `this`

* Clean up InkToolbarCustomPenButton.cs

- Remove `this`

* Clean up InkToolbarCustomToolButton.cs

- Remove unused usings
- Remove `this`
- Remove `global::` alias

* Clean up InkToolbarIsStencilButtonCheckChangedEventArgs.cs

- Remove unused usings
- Simplify names
- Fix property `StencilKind` to use interop type instead of WinRT

* Clean up InkToolbarMenuButton.cs

- Remove unused using
- Remove `this`
- Remove cast to int on `MenuKind`. Enum is already int, so no need to cast then cast to enum.

* Clean up InkToolbarPenButton.cs

- Remove unused using
- Remove `this`

* Clean up InkToolbarStencilButton.cs

- Remove unused usings
- Remove `this`
- Remove `global::` in most places
- Convert `Ruler` to expression bodied member

* Clean up InkToolbarToggleButton.cs

- Remove unused usings
- Remove `this`
- Convert `ToggleKind` to expression body, remove unnecessary cast to int
- Remove `global::` in most places

* Clean up InkToolbarToolButton.cs

- Remove unused usings
- Remove `this`
- Remove most instances of `global::`
- Convert `ToolKind` to expression body, remove unnecessary cast to int

* Clean up BindingBase.cs

- Remove `this`

* Clean up BindingExpression.cs

- Remove `this`

* Clean up BringIntoViewOptions.cs

- Remove `this`

* Clean up Brush.cs

- Remove `this`
- Left review comments for uncaught NRE

* Clean up BrushTransition.cs

- Remove `this`

* Clean up MediaPlayerElement.cs

- Remove unused usings
- Remove `this`

* Leave review comment in MediaSourceConverter.cs

* Remove `Support` folder

Contents moved to WinRT/Interop folder

* Clean up WindowsXamlHostBaseExt.cs

- Add inheritdoc for documentation
- Remove `this`

* Clean up IWebViewCompatible.cs

- Remove unused using
- Group members (all properties, methods, events, etc. together)

* Clean up WebBaseCompatabilityAdapter.cs

- Group members

* Provide Web adapters correct implementation for IDispose pattern

* Clean up WebBrowserCompatabilityAdapter.cs

- Implement proper IDisposable
- Add null checks prior to operations to avoid ObjectDisposeException or NullRefrenceException

* Clean up WebViewCompatabilityAdapter.cs

- Add null checks for operations to avoid unhandled NRE

* Work in progress

* Fixing a merge error.

* updated folder names and updated toolbox xml file

* Add app.config/app.manifest to WinForms sample app

Required to support high DPI

* added visualstudiotoolsmanidest.xml to nuget package

* WIP while updating from master

* Designer now allows association between ink toolbar & its canvas.

* Ink canvas / toolbar updates, dug in on mediaplayerelement.

* MediaPlayerElement oprational to the same level of functionality as WPF edition.

* Project rename fix

* Fixing a reference that broke in merge.

* Ran UpdateHeaders.bat.

* Fixed release build config.

* Adding stub MDs for control documents.

* Add link to ExternDll.cs

* Adding md documentation for wrappers.

* Remove duplicate tag from build props
Move WebBrowserUriTypeConverter to shared

* Adding missing task extensions for both WPF & WinForms.

* Addressing a PR comment about comments.

* Adding no-op Pack targets for the build.

* removed samples apps from building in release mode

* Tweaked docs.

* Fixing TargetPlatformVersion / TargetPlatformMinVersion

* WIP

* Removing code from accidental commit from winforms branch.

* Temp solution to InkToolbar issue.

* set min target version

* Resolves review issues.

* Addressing PR 2299 change requests.

* Cosmetic change to trigger another build to see if it overcomes the OOM error in the previous build.

* updated win32 project to 17733

* updated comments

* removed Shadow.cs because Windows.UI.Xaml.Media.Shadow is removed from latest insider build. Replaced WebViewControl.AddpreLoadedScript with AddInitializeScript

* Clears up event handler on dispose of WidowsXamlHost.

* Renaming packages in #2299 (#2420)

* renamed win32 namespaces

* updated vs toolbox namespaces

* Fix for local build for PR #2490 merged into #2299.

* Added missing projects to Release Build config.

* Revert "Fix for local build for PR #2490 merged into #2299."

This reverts commit d9202e0.

* Update NativeDefines

- Move to correct namespace
- Update to use existing definitions for WM

* Update Win32 code

Move Win32 code into appropriate namespace. Fix issues with `NativeMethods` style classes and add documentation about when to use which and for what.

* Update namespace

- Remove unused namespaces
- Move `using` outside of `namespace` declaration

* Merge branch 'rjmurillo/XamlIslands' into rjmurillo/XamlIslands-ControlTest

* Fix AssemblyInfos for XamlHosts

* API feedback part 1

* API updates #2

* Code docs updates

- Separate remarks into `<remarks />` tag
- Link types with `<see />` tag
- Fix some typos

* Changed the way SetContent works in WindowsXamlHostBase so that it doesn't expose Windows.winmd to the client.

* Code docs updates

- Separate remarks into `<remarks />` tag
- Link types with `<see />` tag
- Fix some typos

* Remove Ryland's test projects from SLN

* Make NativeDefines.cs consistent

New entries added that should use enum `WM`

* Use `var` in WindowsXamlHostBase.KeyboardFocus.cs

* Remove redundant `this` qualifier in WindowsXamlHostBase.KeyboardFocus.cs

* Remove unused `using` in WindowsXamlHostBase.KeyboardFocus.cs

* Fix for SA1505

* Resources designer namespace update

* Prevent _xamlSource from leaking

This field is only needed to surface the content through the `WindowsXamlHost.Child` property and does not otherwise need to be exposed to inherited classes when the property will suffice. To maintain the same API surface as the WPF control, a new API `SetContent` was added to take the `FrameworkElement`. The implementation of `SetContent(FrameworkElement)` is identical to `SetContent(UIElement)` other than the parameter type.

CC @ryalanms

* Work around CoreInputSink bug, Windows Forms Focus issue

* Update SetContent method

Change a6b20f5 introduced a signature change

* Relax WebView fail fast when setting properties in XAML

Remove assert conditions for state and add comments as to what is happening and what expected behavior should be.

* Fixes after pulling in code from Rebased and ControlTest branches.

* Addresses items in PR #2432
nmetulev pushed a commit that referenced this issue Oct 11, 2018
pull latest community toolkit into my fork.
nmetulev pushed a commit that referenced this issue Oct 16, 2018
Update WeiboDataProvider.cs
michael-hawker referenced this issue in michael-hawker/UWPCommunityToolkit Sep 4, 2019
Add Uno support

Temporary adjustments have been added to allow for Uno targets to build or minimally work at runtime.
michael-hawker pushed a commit that referenced this issue Sep 12, 2019
@ghost ghost locked as resolved and limited conversation to collaborators Nov 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug 🐛 An unexpected issue that highlights incorrect behavior wontfix ❌ Discontinued Issues or PR
Projects
None yet
Development

No branches or pull requests

1 participant