Skip to content

Feat/mixer UI#16

Open
smoothdeveloper wants to merge 12 commits intomasterfrom
feat/mixer-ui
Open

Feat/mixer UI#16
smoothdeveloper wants to merge 12 commits intomasterfrom
feat/mixer-ui

Conversation

@smoothdeveloper
Copy link
Copy Markdown
Contributor

No description provided.

@smoothdeveloper
Copy link
Copy Markdown
Contributor Author

@josephlarralde peux-tu regarder les dépendances, genre MainCommandsComponent qui prend le PluginProcessor:

Deux choix:

  • il vaudrait mieux que le panel expose les évenements, et que les évenements soit "bindés" au niveau du plugin processor (qui dépend de tout les composants UI)
  • on peut passer des functors (reset, ou set gain) pour totalement découpler (déjà fait dans PluginProcessor sur onVelocityChanged, ou onEnginePitchChanged, je pense que c'est l'approche la plus flexible)

Tel que c'est, je pense que ça nous donne des dépendences cycliques entre les composants (c'est pas bien du tout pour refactoring, modulariser, etc.).

Si le refactoring se passe bien, faire une revue similaire sur les autres composants pour faire un découplage similaire.

@josephlarralde
Copy link
Copy Markdown
Collaborator

@smoothdeveloper merci pour la suggestion ! je vais regarder ça.

@josephlarralde
Copy link
Copy Markdown
Collaborator

Bon, j'ai bien buggé sur cette tâche et fait quelques détours par CMake et autres travaux de refactoring. Je pense que le découplage des composants JUCE devra passer par un AudioProcessorValueTreeState, ou quelque chose de similaire pour s'abstraire des paramètres concrets. Ma version locale est "en flux" pour l'instant et je ne sais pas s'il sera pertinent de merger cette branche. On en rediscute bientôt.

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