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

feat: table component #788

Merged
merged 27 commits into from
Feb 28, 2024
Merged

feat: table component #788

merged 27 commits into from
Feb 28, 2024

Conversation

bariskaraca
Copy link
Contributor

@bariskaraca bariskaraca commented Jan 30, 2024

Closes #142

Saffet Barış Karaca and others added 2 commits January 8, 2024 14:35
* feat: table components initialized
* feat: added sticky last column
* feat: added full story
@bariskaraca bariskaraca requested a review from ogunb January 30, 2024 06:24
* feat: fix css prettier issue
@leventozen leventozen changed the title Feature/datatable feat: datatable component Jan 30, 2024
@leventozen leventozen changed the title feat: datatable component feat: table component Jan 30, 2024
@buseselvi
Copy link
Contributor

Great job! 🚀 Just, I think sorting works wrong. It looks right on the UI but should be working as ascending (A-Z / 1-9) sorting on the first click. It is working as Z-A / 9-1 in the storybook. Can we fix that? 🙏 @bariskaraca

* feat: add numeric sort to id column in story
src/components/table/bl-table.stories.mdx Outdated Show resolved Hide resolved
src/components/table/bl-table.stories.mdx Outdated Show resolved Hide resolved
src/components/table/bl-table.stories.mdx Outdated Show resolved Hide resolved
src/components/table/bl-table.ts Show resolved Hide resolved
src/components/table/bl-table.ts Outdated Show resolved Hide resolved
src/components/table/table-cell/bl-table-cell.ts Outdated Show resolved Hide resolved
src/components/table/bl-table.ts Outdated Show resolved Hide resolved
bariskaraca and others added 3 commits January 31, 2024 17:55
* feat: Remove console log in story
* feat: Change map to foreach for querySelectorAll
* feat: Change story title to Table
@buseselvi
Copy link
Contributor

When there is a disabled row that can't be selected, the table header checkbox should be indeterminate. We also did that in the Select Component as in the second picture below.

image image

bariskaraca and others added 9 commits February 5, 2024 14:47
* feat: Refactor select all logic
* feat: Edit args table for table stories
* feat: Remove nullish operator for closest
* feat: Add table stories
* feat: Edit sort function in story
* feat: Add tooltip to table header in story
* feat: Edit row selection story texts
* feat: Reset table scroll position on sort
@buseselvi buseselvi self-requested a review February 20, 2024 13:19
Copy link
Contributor

@buseselvi buseselvi left a comment

Choose a reason for hiding this comment

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

  • Sorting icon already exists, it is the same as the sorting_default icon. Can we remove the newly added default one?
image
  • And in the UI Test, it seems there is a problem with the sticky headers and columns in the Firefox browser. Borders of sticky areas are not visible, can we check it?

* feat: add background-clip to cell and header cell
* feat: remove sorting default icon
@AykutSarac
Copy link
Member

There's an issue with the cell corners and width/height:

image

src/components/table/bl-table.ts Outdated Show resolved Hide resolved
src/components/table/bl-table.ts Outdated Show resolved Hide resolved
src/components/table/bl-table.ts Outdated Show resolved Hide resolved
src/components/table/bl-table.ts Show resolved Hide resolved
src/components/table/table-header/bl-table-header.ts Outdated Show resolved Hide resolved
src/components/table/table-cell/bl-table-cell.ts Outdated Show resolved Hide resolved
@leventozen leventozen self-requested a review February 26, 2024 07:22
src/components/table/bl-table.ts Outdated Show resolved Hide resolved
src/components/table/bl-table.ts Outdated Show resolved Hide resolved
src/components/table/bl-table.ts Outdated Show resolved Hide resolved
src/components/table/table-body/bl-table-body.ts Outdated Show resolved Hide resolved
src/components/table/table-cell/bl-table-cell.ts Outdated Show resolved Hide resolved
src/components/table/table-header/bl-table-header.ts Outdated Show resolved Hide resolved
src/components/table/table-row/bl-table-row.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@buseselvi buseselvi left a comment

Choose a reason for hiding this comment

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

I wrote it above but it may have been overlooked: In the UI Test, it seems there is a problem with the sticky headers and columns in the Firefox and Safari browsers. Borders of sticky areas are not visible, can we check it?

image (2)

@buseselvi
Copy link
Contributor

There's an issue with the cell corners and width/height:

image

and can we fix this radius issue, too? 🙏🏻

* feat: fix border issue
* feat: add last child control for shadow left
* feat: remove disabled control for all checked state
Copy link
Member

@leventozen leventozen left a comment

Choose a reason for hiding this comment

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

Thank you to all the contributors and maintainers for your dedication and hard work on the table component. Your efforts have truly elevated our project to the next level. 🚀

@leventozen leventozen merged commit e1c54b9 into next Feb 28, 2024
7 checks passed
@leventozen leventozen deleted the feature/datatable branch February 28, 2024 08:28
Copy link

🎉 This PR is included in version 3.0.0-beta.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

github-actions bot commented May 2, 2024

🎉 This PR is included in version 3.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DataTable Component
6 participants