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

Dev lasso board utilities #596

Draft
wants to merge 7 commits into
base: dev
Choose a base branch
from
Draft

Dev lasso board utilities #596

wants to merge 7 commits into from

Conversation

mlr77
Copy link
Contributor

@mlr77 mlr77 commented May 6, 2023

Added some utilities to the lasso selection to align apps or group assign certain features like a specific color for a stickie or a specific kernel for a group of SAGECells. Most of the functions are written in typescript as an array of updates but these would all be ideal for batch updates. Needs work to rewrite as batchUpdates.

The board.py was adjusted slightly to allow for organizing selected apps as a proof of concept. We can add more "smart functions" as needed.

Screenshot 2023-05-06 at 11 27 06 AM
Screenshot 2023-05-06 at 11 26 48 AM
Screenshot 2023-05-06 at 11 26 43 AM
Screenshot 2023-05-06 at 11 26 32 AM
Screenshot 2023-05-06 at 11 26 24 AM

@mlr77 mlr77 requested review from renambot and rctheriot May 6, 2023 21:32
@mlr77 mlr77 marked this pull request as ready for review May 6, 2023 22:34
@mlr77 mlr77 marked this pull request as draft May 7, 2023 00:53
@renambot
Copy link
Contributor

renambot commented May 7, 2023

  • Nice first pass @mlr77 !
  • Maybe eslint is not loaded in your editor, but a few red marks in my editor (mostly fixed now): const, unused import or variables, any, ...
  • Check the browser console: big error messages
  • Before finishing the PR, maybe create a type for your updates instead of using 'any'.
  • thanks

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