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

@graph-paper/stack: use owl selectors to allow smooth slide in/out transitions instead of grid-column-gap #56

Merged
merged 7 commits into from
Jul 17, 2020

Conversation

hamilton
Copy link
Collaborator

@hamilton hamilton commented Jul 4, 2020

Closes #35

Re-implements the Stack component to use an owl selector instead of grid-column-gap. The stack itself, however, is sitll implemented in CSS grid, giving flexibiilty w/ the same the justify / align props. This allows the spacing to be collapsed in various ways when transitioning in a new element. See the ColorPalette.svelte story as an example.

Practically speaking, this should have no visual effect for end users.

@hamilton hamilton marked this pull request as ready for review July 4, 2020 14:44
openjck referenced this pull request in openjck/graph-paper Jul 7, 2020
See the discussion in #56 for more information.
hamilton added 7 commits July 16, 2020 16:54
…S reset stylesheet first, since it would be good to have * { margin: 0px; } among other rules. This decision actually blocks this PR since we have an issue with explicitly-set 0-margin Button elements not adhering to the stack margin rule.
@hamilton hamilton merged commit 0555e1c into main Jul 17, 2020
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.

@graph-paper/stack: spacing should be dictated by owl selectors + margin, not grid-column-gap
1 participant