- so that both the columnSelector and displaySelector can use it

+ update docs:
  - reorder UI to match toolbar layout
  - fix legends

+ add unit tests (& improve some existing unit tests)

NYI: conditional lineCount controls

+ unit tests

+ fix cell height to dynamically on lineCount change

…efined heights but defined rowHeightsOptions objects

e.g. - empty objs, or lineHeight set but nothing else

+ simplify this.props

…ined mode

- Multiline content includes items with `<br>`/`<p>` tags etc that end up incorrectly vertically centered to their increased height

… + density

- by giving it the same setRowHeight logic as line counts with a single line

+ rename recalculateLineCountHeight for clarity now that it's no longer being used for just lineCount

- to make the difference more obvious when switching between single and custom

…to datagrid-row-height-control

…ering

- expand buttons on single lines should now be more vertically aligned

- The manual button/checkbox changes for control columns is somewhat required for auto height to not be dramatically different for single lines

… rows

- when hovering over rows
- on striped rows

- Not caused by this PR, but this was really annoying me, haha

- Slightly less grody to look at in-code, and allows us to provide more comments for context

The more I look at __expandButton the more I dislike the naming also but it would involve too many changes right now to rename button->actions, so I added a TODO

+ Remove ability to set negative or zero numbers, which causes the grid to flip out

- This should have been done in 79c878d, but I missed it

…ions are disabled

Notes:
- Strongly recommend turning off whitespace changes to reduce indentation noise
- This could have been done at the `data_grid_toolbar` level and could be a microperf optimization to do so, but since this is a relatively unlikely edge case in any case I don't think it's worth the overoptimization when this approach is relatively straightforward to implement

…to datagrid-row-height-control

Co-authored-by: Caroline Horn <549577+cchaos@users.noreply.github.com>

+ not set a button state if custom fontSize/cellPadding don't match our defined density states

…to datagrid-row-height-control

+ improve unit tests to cover undefined nested object key case

…er when using `rowHeights`

@see elastic#5411

Also see https://stackoverflow.com/questions/49384120/resizeobserver-loop-limit-exceeded