lmms icon indicating copy to clipboard operation
lmms copied to clipboard

Make equalizer leg design dependent on theme

Open Monospace-V opened this issue 3 years ago • 4 comments

Enhancement Summary

To make the fader legs on the equalizer input and output the same as those on the fader legs of the FX-Mixer according to theme.

Justification

At the moment, the equaliser's fader appears to be completely independent of the theme, and is based on LMMS' default theme's fader legs regardless of the configuration of FX-Mixer legs according to theme. image It looks inconsistent, and it makes sense to change the design of the EQ-fader based on the theme, along with the FX-Mixer fader.

Monospace-V avatar Jan 06 '22 09:01 Monospace-V

Faders in EQ uses EqFader which is a subclass of Fader. Themes affect Fader, but not EqFader.

EqFader uses a different logic for peak calculation from Fader, that might be the reason there are two classes.

PhysSong avatar Jan 16 '22 02:01 PhysSong

Hi, I have a Software Methodology Development class that requires me to contribute to an open-source repo. Even though this issue does not have a good-first-issue tag, I would like to give it a try.

Sea-Shore-Sahil avatar Jan 26 '22 23:01 Sea-Shore-Sahil

@Sea-Shore-Sahil sorry about the late reply, but you're more than welcome to open a PR fixing this if you're still interested. The classes mentioned by PhysSong, the Fader property in data/themes/default/style.css, and the plugins/Eq folder may all be of interest.

https://github.com/LMMS/lmms/blob/master/plugins/Eq/EqControlsDialog.cpp

Spekular avatar Jan 30 '22 12:01 Spekular

There's a small chance that @michaelgregorius might unify the EqFader with the Fader, from the discussion in #7045, which might fix this one.

Rossmaxx avatar Apr 13 '24 10:04 Rossmaxx