Improve fixed-column layout

[eradman]

What type of PR is this?

• Bug Fix

Description

Only set scroll.x when more than one fixed column is set to preserve text wrapping. Unset .ant-table-content overflow property to ensure virtical scrolling works.

How is this tested?

• Manually

Related Tickets & Documents

#7127

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

fixed_columns.mov

[eradman]

Latest failure

  1) Table
       Fixing columns
         fixes the correct number of columns:

    CypressError: Timed out retrying after 20000ms: `cy.scrollTo()` failed because this element is not scrollable:

`<div class="ant-table-content" style="overflow: auto hidden;">...</div>`

Make sure you're targeting the correct element or use `{ensureScrollable: false}` to disable the scrollable check.

https://on.cypress.io/scrollto

The inline style seems to be set by antd. Overriding this with a stylesheet works, but Cyrpess doesn't like it

[justinclift]

Should we instead revert #7019, and look for a different way to implement it that doesn't break vertical scrolling?

[eradman]

Should we instead revert #7019, and look for a different way to implement it that doesn't break vertical scrolling?

We might have to. It's possible that this behavior was fixed/changed in a later version of antd.

I proposed a revert commit in #7131

[eradman]

The fixed column featurew as reverted in #7131, next step is to propose this feature as a new PR