alex/nih-plug
1
0
Fork 0
Code Activity

Add a minimum width to tooltip width comparison

Browse source 
To make it flicker around less at the right edge of the X-Y pad.
This commit is contained in:
Robbert van der Helm 2022-11-12 01:45:26 +01:00
parent 6f34eb3129
commit d8729ffd34
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
plugins/diopser/src/editor/xy_pad.rs
View file

@ -302,7 +302,12 @@ impl View for XyPad {
.get_last_child(cx.current())
.expect("Missing child view in X-Y pad");
let tooltip_bounds = cx.cache.get_bounds(tooltip_entity);
self.tooltip_pos_x = if (relative_x + tooltip_bounds.w + 4.0) >= bounds.w {
// NOTE: The width can vary drastically depending on the frequency value, so we'll
// hardcode a minimum width in this comparison to avoid this from jumping
// around. The parameter value updates are a bit delayed when dragging the
// parameter, so we may be using an old width here.
self.tooltip_pos_x = if (relative_x + tooltip_bounds.w.max(150.0) + 4.0) >= bounds.w
{
Pixels(relative_x - 2.0 - tooltip_bounds.w)
} else {
Pixels(relative_x + 2.0)