TCP FX list scroll bug (FIXED in TCP, but not in MCP)

FeedTheCat · 12-04-2020, 03:17 AM

Scroll behavior is broken when minimum column width is smaller than maximum column width (param 3&4 of tcp.fxparm.font). You can scroll one tick more than you should be able to.

You can quickly check the behavior in my new theme mod.

FeedTheCat · 04-02-2021, 04:39 AM

Bumping this as there's more and more themes using TCP FX one the right side.

vitalker · 04-02-2021, 07:12 AM

Would be nice to have this fixed.

Justin · 04-04-2021, 07:50 PM

Fixing for 6.27!

FeedTheCat · 04-05-2021, 10:43 AM

Thank you, working great now!

FeedTheCat · 03-31-2022, 01:18 PM

Quote:

Originally Posted by Justin

Fixing for 6.27!

MCP FX list has the same issue, maybe it's trivial to apply the same fix?

FeedTheCat · 03-31-2022, 01:44 PM

Quote:

Originally Posted by vitalker

You know you can delete posts?

What post?

Thanks, that's handy, didn't know.

vitalker · 03-31-2022, 01:45 PM

Quote:

Originally Posted by FeedTheCat

What post?

Thanks, that's handy, didn't know.

Justin · 04-01-2022, 08:00 PM

Quote:

Originally Posted by FeedTheCat

MCP FX list has the same issue, maybe it's trivial to apply the same fix?

Link to that theme?

In general using multiple columns for the FX list in the MCP is a bad idea -- they are variable size due to embedded UI possibilities, really should keep it to a single column.

FeedTheCat · 04-02-2022, 01:12 AM

Quote:

Originally Posted by Justin

Link to that theme?

Here's a link to a modded default theme:

I only changed param 3 of mcp.fxlist.font to 32

Quote:

Originally Posted by Justin

In general using multiple columns for the FX list in the MCP is a bad idea -- they are variable size due to embedded UI possibilities, really should keep it to a single column.

Makes sense, the theme I'm using is doing it by default (it doesn't have a limit of columns though, so I modded it to have 2 columns at max). I didn't like it in the beginning, but I've grown to appreciate being able to see more fx at one glance. Then again, I barely use the embedded FX UI feature.

Anyways, I was thinking the fix would be identical to the one you already applied for the TCP. If it's more work, don't bother.

Thanks!

FeedTheCat · 04-02-2022, 04:51 AM

I just uploaded my mod of the Echolot theme if you're interested (the one in the GIF). There's a setting in the theme adjuster that allows users to specify the max allowed columns for the MCP fx-list. I heeded your advice though and set the default to 1. The scroll bug then occurs when the max columns setting is set to 2 or more.