Max list safe length too high in variable-manager

[valmontechno]

Description

The variable-manager addon has a maximum length for the lists it can display.

ScratchAddons/addons/variable-manager/userscript.js

Line 99 in 724f29b

maxSafeLength = 5000000;

However, this limit is higher than the natural scratch limit.

In Scratch 3.0, lists have a hard size limit of 200,000 items.

So the list too big value message can't be displayed.

Steps to reproduce

scrnli_01_05_2024.12-57-25.webm

Demonstration(s) and additional context

No response

Environment

Browser: Google Chrome 124.0.6367.63
Scratch Addons version: v1.37.1
Operating system: Windows 10

[mxmou]

maxSafeLength is the maximum length of the string representation of the list, not the maximum number of items.

[valmontechno]

I understand, this is the maximum number of characters.