search

thuliteio / create-thulite

Official CLI for scaffolding a new Thulite project
Other
0 stars 1 forks source link

Increase default `searchLimit` #14

Closed salim-b closed 1 year ago

salim-b commented 1 year ago

Was there a specific reason for the low former limit of 5?

h-enk commented 1 year ago

It was a UX thing before the scrollbar implementation — but that's no longer relevant. So, I guess it's OK to set it to 99 per default.

Not sure though what it means in terms of performance. Maybe partly aside, but I'd like to run a little test with a large number of pages to see how FlexSearcrh holds in the new setup (see also https://github.com/h-enk/doks/discussions/859)

h-enk commented 1 year ago

Note to self: Adding many pages results in "RangeError: Maximum call stack size exceeded"