adaptive-table recursion level limited to 2 ?

[Ognian]

Hi,
is there a reason why the recursion level is limited here?
https://github.com/geraintluff/jsonary/blob/master/renderers/contributed/adaptive-table.js#L86

I have deeper levels, so I've removed this…
Is it OK for you to remove this check?

Ognian

[geraintluff]

There has to be some limit, but it doesn't have to be 2 levels.

What happened was that I attempted to render a JSON Schema object - this is, of course, recursive, so it got locked in a loop, creating columns for subschemas of subschemas, etc.

The level could easily be raised - or perhaps specified using a displayAsTableDepth parameter?

[Ognian]

You are right, a displayAsTableDepth parameter would be OK, but if this limit is reached we should also somehow show this to the user, like "nesting too deep" instead of the link. The default level should be something like 5 or 10, what do you think?

[geraintluff]

Ah - collapsing the display, instead of rendering the data in non-array mode. That's a good idea. :)