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.

Sign up for GitHub

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

Jump to bottom

Snapping to Common Distances #484

Closed
jauyong opened this issue Mar 5, 2020 · 5 comments · Fixed by #884
Closed

Snapping to Common Distances #484

jauyong opened this issue Mar 5, 2020 · 5 comments · Fixed by #884
Assignees
@wassgha

Comments

@jauyong
Copy link

jauyong commented Mar 5, 2020 •
edited

Feature Description

As and author I want to snap to common distances based on other elements on the page for a consistent look

Please see this video on slack. .mov is not allowed in Github.

image

Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance Criteria

  1. Snapping to Common Distances
    Note: multi-selection is treated as a single element.

QA Instructions

  1. You need 3 equal size elements
  2. Position 2 next to each other
  3. Take the third and try to position it with teh same distance between the first 2 elements.
@wassgha wassgha mentioned this issue Mar 5, 2020
Merged
@jauyong
Copy link
Author

jauyong commented Mar 5, 2020

Closing this. See PR #488 and #285

@jauyong jauyong closed this as completed Mar 5, 2020
@jauyong jauyong reopened this Mar 5, 2020
@jauyong
Copy link
Author

jauyong commented Mar 5, 2020

@samitron7 to add screenshots

@jauyong jauyong added the Size: M label Mar 5, 2020
@kmyram kmyram added this to the Sprint 25 milestone Mar 9, 2020
@jauyong
Copy link
Author

jauyong commented Mar 13, 2020

@samitron7 I know this ticket is closed but you mentioned you added a screencast link here but I don't see it.

@jauyong
Copy link
Author

jauyong commented Mar 17, 2020

I have linked the slack message containing the .mov and added the screenshot to the issue.

@jauyong jauyong removed the UX Needed label Mar 17, 2020
@wassgha
Copy link
Contributor

wassgha commented Mar 30, 2020

Tracking daybrush/moveable#200

@wassgha wassgha mentioned this issue Mar 30, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wassgha wassgha

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Snap to common element distances GoogleForCreators/web-stories-wp
4 participants
@wassgha @kmyram @jauyong @samitron7