Replace remaining "legacy" knobs with enhanced ones
- Dominant language
- C++
- Stars
- 10.4k
- Forks
- 1.3k
- Avg merge
- 2d 13h
- Merged PRs (30d)
- 7
Description
Pull request https://github.com/LMMS/lmms/pull/7525 has introduced the option to individually set the font sizes for knob labels by making them adhere to the font size that's set in the widgets font. It also allows to set the font sizes of individual knobs via CSS.
Due to layout constraints it was not possible to introduce this new feature in all places and some knobs still have to fall back to legacy rendering of the labels. Attempting to fix all these places would have been too much scope and too much work for the aforementioned PR. Hence it should be attempted to fix these remaining places one after the other.
The following gives an overview with images of the remaining places. You will find that most of the problems are caused by missing layouts, hard-coded positions and insufficient space provided by the encompassing parent widget.
## Delay plugin

This plugin might need some rework in general as the knobs do not align.
## EffectView

## EnvelopeAndLfoView

This view looks very crowded and should get more space to present its widgets.
## InstrumentFunctionViews

## InstrumentSoundShapingView

## InstrumentTrackView

This view might benefit from a bit more vertical space.
## SampleTrackView

This view might benefit from a bit more vertical space.
# Other places
The Carla plugin also uses the legacy knobs. I do not have Carla installed and therefore cannot provide a screenshot. This is likely also the reason why it was not adjusted in the PR in the first place. It might even be a low-hanging fruit for any developer who has Carla installed.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by locating the Delay plugin, EffectView, EnvelopeAndLfoView, InstrumentFunctionViews, InstrumentSoundShapingView, InstrumentTrackView, SampleTrackView, and Carla plugin UI implementations. Compare their layouts and knob rendering with the enhanced implementation introduced by pull request #7525. Done means the remaining legacy knobs use the enhanced rendering without breaking the views' layout; verify each affected view visually.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- design, desktop
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100