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

Initial 'Effects' component + AttachedShadowBase/AttachedDropShadow #19

Merged
merged 7 commits into from
Apr 19, 2023

Conversation

michael-hawker
Copy link
Member

@michael-hawker michael-hawker commented Apr 13, 2023

Related to #5

Goal for the 'Effects' package is to hopefully house anything related to Composition and tweaking visuals in a nice package that uses the similar Effects.* attaching API (we could potentially do this with the Pipeline Brush and Visual Factory stuff in the Media package, at least for anything that doesn't use Win2D effects (which is minimal, but still abstractions being here could be good)). We'd have a similar Effects.Media or Media.Effects for the Win2D dependency with more implementations...

This package only supports UWP and WinAppSDK. (We still have the issue in the individual component solution with the WASM head being brought over.) Related to CommunityToolkit/Tooling-Windows-Submodule#5

Ported from WinUI branch, audited against UWP branch. Should be good outside of bringing over more samples and better breaking them out in the docs to explain style usage vs. shared as a resource vs. adding to a naked element.

@michael-hawker
Copy link
Member Author

Oops, mis-renamed a file/type in the test, should hopefully be good now.

@michael-hawker michael-hawker linked an issue Apr 18, 2023 that may be closed by this pull request
Tested on WinAppSDK + UWP (need to compare with UWP branch next though)
Removed need for internal DesignTimeHelpers in Extensions (only cares about 16299 and lower which is below our min target)
Added basic sample, needs to bring over more detail/doc here
Added overall doc from doc repo, though needs editing/updating
@michael-hawker
Copy link
Member Author

Rebased on latest main

Co-authored-by: Arlo <arlo.godfrey@outlook.com>
@michael-hawker michael-hawker added this pull request to the merge queue Apr 19, 2023
Merged via the queue into main with commit 23e5e86 Apr 19, 2023
@delete-merged-branch delete-merged-branch bot deleted the port/shadows branch April 19, 2023 05:53
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.

Port Shadows
2 participants