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

Rework from Size animations to Scale animations, add StackAlignment + StackDensity #45

Closed
wants to merge 7 commits into from

Conversation

thomas-stockx
Copy link

@thomas-stockx thomas-stockx commented Feb 1, 2022

This PR contains several changes:

  • Rework to use Scale instead of Size for resizing middle and back cards.
    The current Size method worked fine for images, but when embedding text in cards it caused issues do to text wrap as the container was resizing. I've reworked the resizing to use Scale animations instead.

  • Added Stack Alignment & StackDensity
    In the current implementation there's no way to change how the stack looks. Cards are always on top of each other, with the middle and back cards sticking out at the bottom of the widget.
    With StackAlignment you can change where the middle and back cards peek out.
    With StackDensity you can change how much the middle and back cards stick out.

I've tried to maintain as much similarity to the old (Size) resizing as possible, but it might introduce minor visual changes, so I would flag this as a breaking change.

@thomas-stockx thomas-stockx changed the title Add a stackAlignment parameter to customize the alignment of the stac… Add StackAlignment + StackDensity Feb 1, 2022
@thomas-stockx thomas-stockx changed the title Add StackAlignment + StackDensity Rework from Size animations to Scale animations, add StackAlignment + StackDensity Feb 1, 2022
@thomas-stockx
Copy link
Author

I'm closing this PR as it as deviated from the original package by a substantial amount and would cause too many breaking changes in dependend apps/packages.

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.

None yet

2 participants