-
Notifications
You must be signed in to change notification settings - Fork 18
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
[heuristic-table-plugin]table-row/collum styling #33
Comments
@xbot13 Certainly an interesting feature, but not as easy as it might seem. Styling rowsBecause of An other issue is width computation. Each column width is computed through inspecting the content. If one would add padding to rows, this should be taken into account during width computation, while also taking into account virtualization. One solution could be to only style the top-level virtual row, and extract any box model style for width computation. We might also want to prevent usage of specific styles such as horizontal margins, (min,max)width and (min, max)height... Styling columnsThat looks pretty hard since "column containers" are nested in rows. Workaround?You might be able to reach the same visual effect by styling individual cells conditionally. See how to achieve this here.. I'm totally open to PRs regarding this feature, but I don't have the bandwidth right now to implement this! |
First of all thanks for the quick response! I appreciate the library and your work. Honestly I did not think about the But I will definitely have a go at the workaround. And in case someone faces a similar problem, I will post the result here. Can't wait for the stable release! |
@xbot13 Thank you so much for your feedback. Looking forwards to your findings! |
@xbot13 An alternative would be to export two components / split this lib in two, with a lot of the shared logic packed into a third lib (it's a monorepo, so easy to do). And one of these would not support cell spanning but allow line styling. But it's quite an endeavor and I'm not going to dive into it unless for a paid freelance gig! However, I'm still open to PRs. |
Oath
I swear that I have completed these tasks before submitting:
Decision table
official guidelines and recommendations, third-party deprecations,
refactoring or development tooling
Feature
Summary:
2 Props for injecting styling for row and/or collum of the table. As tagsStyles: { tr: {}} has no effect on style.
Idea:
2 new Props for TableRenderer, wich receive viewStyleProps for styling the TableFlexRowContainer/TableFlexCollumContainer respectively. This is especially usefull for styles like shadows that have to apply to the container.
example prop:
example output:
The text was updated successfully, but these errors were encountered: