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

AnalyticalTable: tree table does not render components in cell #358

Closed
stevetsanders opened this issue Mar 17, 2020 · 3 comments · Fixed by #359
Closed

AnalyticalTable: tree table does not render components in cell #358

stevetsanders opened this issue Mar 17, 2020 · 3 comments · Fixed by #359
Assignees

Comments

@stevetsanders
Copy link

Describe the bug
Creating a tree table does not render anything like Button's, Text's, etc in any cell. Only data accessors are shown

To Reproduce
Steps to reproduce the behavior:
https://codesandbox.io/s/dawn-http-e7o19

Expected behavior
I expect the Text components to be rendered inside of the first Actions column. I have two actions columns to demonstrate that it does not work in any column of the tree table. I wanted to use Buttons but codepen was not compiling the code correctly when I used them -- so I commented them out for now.

It works in a non-tree table as also shown in the codepen.

UI5 Web Components for React Information
Operating System:
@ui5/webcomponents-react version: 0.8.6

@MarcusNotheis
Copy link
Contributor

Hi @stsander07
the fix is released in @ui5/webcomponents-react@0.8.7-dev.12

@stevetsanders
Copy link
Author

Hi Marcus,

After upgrading I am receiving this error:

Failed to compile.

./node_modules/@ui5/webcomponents-react/lib/ThemeProvider.js
Module not found: Can't resolve '@ui5/webcomponents-react-base/lib/ThemingParameters' in 'C:\Users\I858355\Documents\1 - Code\ibp-react-ui\node_modules@ui5\webcomponents-react\lib'

Do you know if there is an issue with the build?

@MarcusNotheis
Copy link
Contributor

Hi @stsander07, if you update to 0.8.7-rc.0 the build error should be fixed and you can remove the resolutions field we talked about yesterday.

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 a pull request may close this issue.

2 participants