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

Add Kill buttons to Butterworth8EQEffect #297

Closed

Conversation

badescunicu
Copy link
Contributor

No description provided.

@daschuer
Copy link
Member

Thank yo Nicu. It works good with shade_dev skin.
IMHO this is a 1.12 candidate, what do others think?
It depends on the skin, if it features kill buttons in the effect rack,

@ywwg
Copy link
Member

ywwg commented Jul 24, 2014

Is this building on Effect Button Parameters (281)? @rryan had some big reservations about that PR so I'm wary of plowing ahead so quickly without getting his input.

@badescunicu
Copy link
Contributor Author

Yes, it is based on that PR.

@daschuer
Copy link
Member

@ywwg: This topic is getting worse since Nicu is working on a solution for enum parameters based on this. We have discussed:

  • One parameter type that includes Knobs Buttons and Combo-Boxes
  • Three parameter arrays for each type
  • Two parameter arrays one for Knobs and one for pushbutton features muti-state toggle and a kind flyout combobox.

Any input is welcome but he need it asap to be issued.

@ywwg
Copy link
Member

ywwg commented Jul 24, 2014

why is everything "asap"? I don't like being rushed, especially when I feel like you're trying to rush in features that you know other people disagree with.

@kain88-de
Copy link
Member

@rryan any objections to this?

@daschuer daschuer mentioned this pull request Oct 7, 2014
@kain88-de
Copy link
Member

is this PR still valid after the recent kill button PR made by @daschuer ?

@daschuer
Copy link
Member

daschuer commented Nov 7, 2014

We can close it.

@daschuer daschuer closed this Nov 7, 2014
m0dB pushed a commit to m0dB/mixxx that referenced this pull request Jan 21, 2024
…ase-announcement

blog(release): initial draft Mixxx 2.4 beta release
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.

4 participants