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
fix: refactor table to remove custom elements #689
Conversation
selector: 'fd-table-body', | ||
// TODO to be discussed | ||
// tslint:disable-next-line:component-selector | ||
selector: '[fd-table-body]', | ||
templateUrl: './table-body.component.html' | ||
}) | ||
export class TableBodyComponent {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this will fix the issue good stuff
These components should be directives though, no? Files should be renamed as well in that case
</thead> | ||
<ng-content select="fd-table-body"></ng-content> | ||
<ng-content select="thead"></ng-content> | ||
<ng-content select="tbody"></ng-content> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe add an ng-content for custom stuff? might as well
as mentioned I think you can remove all the css in the tables scss file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
blocking till the date picker fix is out
(possibly) fixes #396
can't reproduce the issue on my machine so we will have to ask around and see if this fixes it.