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

feat: allow the user to move the board #257

Conversation

beawar
Copy link
Contributor

@beawar beawar commented May 31, 2023

  • fix: propagate local storage updates globally to each hook usage

refs: SHELL-27 SHELL-99

beawar added 22 commits May 12, 2023 16:57
refs: SHELL-98 SHELL-27
Use last saved sizes as dimensions of a new board.
If there is no saved size, use defaults.
Update defaults to be responsive and to respect 4/3 aspect ratio.

refs: SHELL-98 SHELL-97
fix: prevent board from moving when resizing under min size

refs: SHELL-27 SHELL-98 SHELL-102
refs: SHELL-99 SHELL-27
Prevent move of new board to reset size by initializing the internal ref
with the local storage value.

Prevent board position to be calculated when collapsed, since the board container
has size 0 and the calculation will result in a negative value.

Remove listener of the click on mouseUp to always clean it up

refs: SHELL-99
@beawar beawar requested review from a team as code owners May 31, 2023 13:02
@beawar beawar requested review from nubsthead, giuliano176 and AlessioCrestani and removed request for a team May 31, 2023 13:02
Click event can be fired after the mouse up, so removing the listener on
mouse up is causing the click to be triggerd after a move.

refs: SHELL-99
@beawar beawar requested review from nubsthead, giuliano176, AlessioCrestani and a team and removed request for a team, nubsthead, giuliano176 and AlessioCrestani May 31, 2023 13:58
@beawar beawar self-assigned this May 31, 2023
Base automatically changed from SHELL-98-allow-the-user-to-resize-the-board to SHELL-27-resize-and-move-board June 5, 2023 17:16
@beawar beawar merged commit b99476e into SHELL-27-resize-and-move-board Jun 6, 2023
@beawar beawar deleted the SHELL-99-allow-the-user-to-move-the-board branch June 6, 2023 10:22
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