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

Decouple logic from interactive grid component #1

Merged
merged 2 commits into from
Oct 31, 2024
Merged

Conversation

nkzarrabi
Copy link
Owner

Fixes nicolasperez19#16

Decouple logic from the interactive grid component.

  • Add src/utils/micLogic.ts to export functions for handling microphone positions, zoom, pan, modes, mouse interactions, coordinate conversions, and numpy array string generation.
  • Modify src/components/MicMasterFlex.tsx to import functions from src/utils/micLogic.ts and replace state management, event handlers, and numpy array string generation with utility functions.
  • Add src/utils/micLogic.test.ts to test handling microphone positions, zoom, pan, modes, mouse interactions, coordinate conversions, and numpy array string generation.

For more details, open the Copilot Workspace session.

Fixes nicolasperez19#16

Decouple logic from the interactive grid component.

* Add `src/utils/micLogic.ts` to export functions for handling microphone positions, zoom, pan, modes, mouse interactions, coordinate conversions, and numpy array string generation.
* Modify `src/components/MicMasterFlex.tsx` to import functions from `src/utils/micLogic.ts` and replace state management, event handlers, and numpy array string generation with utility functions.
* Add `src/utils/micLogic.test.ts` to test handling microphone positions, zoom, pan, modes, mouse interactions, coordinate conversions, and numpy array string generation.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/nicolasperez19/mic-master-flex/issues/16?shareId=XXXX-XXXX-XXXX-XXXX).
@nkzarrabi nkzarrabi merged commit 2edf21c into main Oct 31, 2024
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.

Decouple logic from interactive grid component
1 participant