Deprecate old flanger and filter

Bug #1306223 reported by Owen Williams
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Low
Unassigned

Bug Description

We shouldn't expose the old flanger and filter any more. Preferably I think we should remove the COs completely.

(this is a problem now if the user is in midi learn mode and clicks the flanger in the Mixxx UI)

Revision history for this message
Max Linke (max-linke) wrote : Re: [Bug 1306223] [NEW] Deprecate old flanger and filter

This would also break compatibility to old midi-mappings. I'm not sure
how backwards compatibility looks right now with midi-mappings but I
thought it was the same like with the config/database. There an new
config version will also work with an old version of mixxx.

On Thu, 2014-04-10 at 19:11 +0000, Owen Williams wrote:
> Public bug reported:
>
> We shouldn't expose the old flanger and filter any more. Preferably I
> think we should remove the COs completely.
>
> (this is a problem now if the user is in midi learn mode and clicks the
> flanger in the Mixxx UI)
>
> ** Affects: mixxx
> Importance: Undecided
> Status: New
>

Revision history for this message
Daniel Schürmann (daschuer) wrote :

Flanger and Filter CO are already removed. So it does not make sense to allow to map them.
Probably, we will reintroduce a dedicated filter knob, if we move the EQ to the effect domain. But this is a 1.13 issue.

Changed in mixxx:
status: New → Confirmed
Revision history for this message
Owen Williams (ywwg) wrote :

In my midi learn PR, I added an error message if the user tries to map a deprecated control.

Revision history for this message
Max Linke (max-linke) wrote :

Owen is this fixed for you with the midi-learn changes? I'm good if we warn a user that he is mapping deprecated controls.

Revision history for this message
Owen Williams (ywwg) wrote :

yes the warning works.

Revision history for this message
Owen Williams (ywwg) wrote :

The next step of this bug is to remove these COs from any skin we ship with 1.12 -- so I will leave it open.

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

The COs aren't in our 3 skins so I think we can call it fixed now.

Changed in mixxx:
importance: Undecided → Low
status: Confirmed → Fix Committed
Revision history for this message
jus (jus) wrote :

So all "[ChannelN],filter$" controls can safely move to the Deprecated controls section in http://www.mixxx.org/wiki/doku.php/mixxxcontrols

We should document which new controls have to be used instead In order to achieve this functionality.

RJ Skerry-Ryan (rryan)
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/7412

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.