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

Bugfix: Table "no rows to show" - text alignment fixed + continued enhancements for advanced table #988

Open
wants to merge 51 commits into
base: master
Choose a base branch
from

Conversation

verena-ifx
Copy link
Contributor

@verena-ifx verena-ifx commented Feb 6, 2024

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---

CONTRIBUTING.md also tells you what to expect in the PR process.

Description
Text alignment fixed
Table demonstration in example apps fixed

📦 Published PR as canary version: 21.9.1--canary.988.3f82503589b17b2da5b3b8f7fe6f0d43e5afc438.0

✨ Test out this PR locally via:

npm install @infineon/infineon-design-system-stencil@21.9.1--canary.988.3f82503589b17b2da5b3b8f7fe6f0d43e5afc438.0
# or 
yarn add @infineon/infineon-design-system-stencil@21.9.1--canary.988.3f82503589b17b2da5b3b8f7fe6f0d43e5afc438.0

@verena-ifx verena-ifx linked an issue Feb 6, 2024 that may be closed by this pull request
@verena-ifx verena-ifx added the patch patch version bump label Feb 6, 2024
@verena-ifx verena-ifx self-assigned this Feb 6, 2024
Copy link
Contributor

github-actions bot commented Feb 6, 2024

PR Preview Action v1.4.7
🚀 Deployed preview to https://Infineon.github.io/infineon-design-system-stencil/pr-preview-react-example/pr-988/
on branch gh-pages at 2024-05-27 08:14 UTC

Copy link
Contributor

github-actions bot commented Feb 6, 2024

--STORYBOOK-PREVIEW--

@verena-ifx verena-ifx changed the title Bugfix: Table "no rows to show" - text alignment fixed Bugfix: Table "no rows to show" - text alignment fixed + continued enhancements for advanced table Feb 12, 2024
@tishoyanchev
Copy link
Contributor

  • What is the purpose of the time showing when no rows or cols?
  • Can you create internal states for columnDefs, rowData?
  • Can you move gridApi into componentWillLoad to prevent risk of infinite re-renders?
  • Should we not include the advanced table in the example apps?
  • In React, when I remove rows, I get an error: Unexpected value for cols and rows: undefined ... If I remove
    the cols, I get: Unexpected value for cols and rows: .. and Unexpected value for cols: undefined

@verena-ifx verena-ifx linked an issue Feb 18, 2024 that may be closed by this pull request
verena-ifx added 29 commits April 25, 2024 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch patch version bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Text inside table collides with table-header Set Filter for Table component
2 participants