Removed the space between the thead and tbody of the Table Component.

[kboeh]

Current Behavior

This fix affects the react-table package. There is a space between the thead and tbody elements in the table component.

New Behavior

I added 'border-collapse: collapse' property in order to remove the space between the thead and tbody.

Related Issue(s)

Fixes #24439

[ghost]

All CLA requirements met.

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 18b1ee8:

Sandbox	Source
@fluentui/react 8 starter	Configuration
@fluentui/react-components 9 starter	Configuration

[fabricteam]

📊 Bundle size report

Unchanged fixtures

Package & Exports	Size (minified/GZIP)
react-components react-components: Accordion, Button, FluentProvider, Image, Menu, Popover	187.656 kB 51.96 kB
react-components react-components: FluentProvider & webLightTheme	33.359 kB 11.004 kB
react-portal-compat PortalCompatProvider	5.851 kB 1.964 kB

_{🤖 This report was generated against 83b033de0fe819bc30cc437ec7c4a83dd4c6997e}

[size-auditor]

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 83b033de0fe819bc30cc437ec7c4a83dd4c6997e (build)

[fabricteam]

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario	Render type	Master Ticks	PR Ticks	Iterations	Status
Avatar	mount	1734	1648	5000
Button	mount	1251	1236	5000
FluentProvider	mount	1883	1914	5000
FluentProviderWithTheme	mount	718	705	10
FluentProviderWithTheme	virtual-rerender	651	671	10
FluentProviderWithTheme	virtual-rerender-with-unmount	716	699	10
MakeStyles	mount	2259	2237	50000
SpinButton	mount	3302	3259	5000

[ling1726]

Thanks for the contribution @kboeh 🙏. Unfortunately, there are big changes upcoming in #24762 that are going to be targeted for the next preview release of the table component. The border-collapse styling will be there

[kboeh]

Thanks for the contribution @kboeh 🙏. Unfortunately, there are big changes upcoming in #24762 that are going to be targeted for the next preview release of the table component. The border-collapse styling will be there

No problem. This was my first ever open-source PR and I learned allot. Maybe next time! Do I simply close this PR myself now?

[ling1726]

Thanks for the contribution @kboeh 🙏. Unfortunately, there are big changes upcoming in #24762 that are going to be targeted for the next preview release of the table component. The border-collapse styling will be there

No problem. This was my first ever open-source PR and I learned allot. Maybe next time! Do I simply close this PR myself now?

Yup feel free to close the PR, I can do that too if you can't. Feel free to keep contributing! It's just unlucky that this contribution is for something under heavy development on our side at the same time