I guess I could go either way on this.

What I was originally trying to avoid was setting an arbitrary `id` and having to increment that every time a new legend string was added. I can't just decrement the counter because I'm currently allowing users to delete any legend string they want, so they might delete `id=3` and have legend strings with ids `1,2,4,5`, and I didn't want to write code to manage that. I also didn't want widgets to have legend strings with `id=74` or something similarly large.

The easiest solution (in my mind, at the time) was to limit them to 5 strings and just hold on to those in the `dashboard_json`. Between removing the standard `splice` logic we've been using and most dashboards only using 1 or 2 expressions, 5 legend strings seemed like it would be enough.

What are your thoughts on this?