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

Fix linear moves #10

Merged
merged 3 commits into from
Aug 26, 2024
Merged

Fix linear moves #10

merged 3 commits into from
Aug 26, 2024

Conversation

knuton
Copy link
Member

@knuton knuton commented Aug 26, 2024

Fixes an issue with linear groups, where expansive items were at a disadvantage.

Due to an implementation error that does not matter much as long as all items have similar size or appear in linear order.

Broken:

linear-stretch-fixed-2024-08-26_16.21.39.mp4

Fixed:

linear-stretch-fixed-2024-08-26_16.18.45.mp4

@knuton knuton force-pushed the fix-linear-moves branch 3 times, most recently from 4df6ac7 to 3b7fa93 Compare August 26, 2024 14:03
The implementation incorrectly inverted the direction of movement for
both the origin point and element annotation, but it should only be done
for the origin point.
@knuton knuton marked this pull request as ready for review August 26, 2024 14:22
@knuton knuton requested a review from stoeffel August 26, 2024 14:23
@stoeffel
Copy link
Contributor

How did you discover this? Was there an issue on our page?

knuton added a commit to knuton/playos that referenced this pull request Aug 26, 2024
This is vendoring focus-shift from a bugfix PR:
dividat/focus-shift#10

Should update reference when bugfix is merged.
@knuton
Copy link
Member Author

knuton commented Aug 26, 2024

How did you discover this? Was there an issue on our page?

I am adding focus-shift to the PlayOS controller, and it caused an issue with the sidebar there. Pressing up or down on a freshly loaded info page jumped to the remote maintenance button, instead of to the sidebar.

Copy link
Contributor

@stoeffel stoeffel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looking good to me! 🎉

@stoeffel stoeffel merged commit 2956641 into dividat:main Aug 26, 2024
6 checks passed
@knuton knuton deleted the fix-linear-moves branch August 26, 2024 14:41
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.

2 participants