[EuiDataGrid] regression in displaying Arrays and Objects in expand behavior

[mmourafiq]

It seems like a regression was introduced to the datagrid expand behavior, it adds Row: 3, Column: 6: which was not the case before.

• Json objects can be tested on this docs page:

• Arrays before regression:

• Arrays after regression:

[cchaos]

Looks like the screen-reader text is accidentally being displayed. I'm thinking that because it's probably getting included within the EuiCodeBlock, it's screen-reader only classes are getting stripped. I'll take a look. Thanks for the report!

[cchaos]

Looks like #4853 added some wrappers and screen-reader text directly to the cell contents. This contents is what's passed to the popover content and depending on the schema (for code) it all gets wrapped by EuiCodeBlock.

@VladLasitsa Can you take a look at this? I'm unsure what the extra wrappers and screen-reader text is necessary for / where it can be moved to instead.

[cchaos]

It also looks like these code blocks were formatted better before that PR as well.

Before

After

[snide]

Looked at this one some. @cchaos has the problem nailed. This is primarily due to the content using innerText which strips out the selectors from the screenreader. Because innerText removes the indentation, this is why the formatting comes in worse.