Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: don't wrap the indexes table header cells #6276

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

lerouxb
Copy link
Contributor

@lerouxb lerouxb commented Sep 24, 2024

I couldn't look at this anymore

before:
Screenshot 2024-09-24 at 13 45 55

after:
Screenshot 2024-09-24 at 14 00 07

@@ -126,7 +127,7 @@ export function IndexesTable<T>({
isSticky
className={cx(tableHeadStyles, darkMode && tableHeadDarkModeStyles)}
>
{table.getHeaderGroups().map((headerGroup: HeaderGroup<T>) => (
{table.getHeaderGroups().map((headerGroup) => (
Copy link
Contributor Author

@lerouxb lerouxb Sep 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a drive-by to get rid of the typescript error that shows in vscode.

@lerouxb lerouxb force-pushed the indexes-table-header-nowrap branch from 039654f to c6a6e2d Compare September 24, 2024 13:03
@lerouxb lerouxb merged commit 8766574 into main Sep 24, 2024
23 of 25 checks passed
@lerouxb lerouxb deleted the indexes-table-header-nowrap branch September 24, 2024 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants