euphoria-io / heim

A real-time community platform.
https://euphoria.io
Other
874 stars 47 forks source link

Avoid crashes with deep threads #103

Open CylonicRaider opened 7 years ago

CylonicRaider commented 7 years ago

Although the Euphoria-side thread depth limit has fallen, React still starts behaving badly when too deeply nested messages appear, leading to looping crashes in certain situations (e.g. showAllReplies enabled or a very deep thread the user posted in), potentially rendering Euphoria entirely unusable. This avoids the problem by forcing a collapse (disregarding showAllReplies) at a (large) depth as a “safety net” against the crashes. The user can still risk seeing the remainder of the thread by expanding the collapsed reply section.