-
Notifications
You must be signed in to change notification settings - Fork 0
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: table-component #37
Conversation
src/components/index.ts
Outdated
//Form | ||
export { default as Input } from './Form/Input/Input'; | ||
export { default as TextArea } from './Form/TextArea/TextArea'; | ||
export { default as CheckBox } from './Form/CheckBox/CheckBox'; | ||
export { default as Select } from './Form/Select/Select'; | ||
export { default as Radio } from './Form/Radio/Radio'; | ||
export { default as LoadingWrapper } from './Form/LoadingWrapper/LoadingWrapper'; |
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.
??
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.
bhai banda use kaise karega agar apun export nhi marenge to
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.
done
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.
but what is this loading wrapper for ? I dont see its usage in Table !
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.
wo export krna bhul gaye the
import React from 'react'; | ||
import { TableCellProps } from './types'; | ||
|
||
const TableCell: React.SFC<TableCellProps> = ({ children }) => { |
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.
FC
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.
done
import React from 'react'; | ||
import { TableHeadingProps } from './types'; | ||
|
||
const TableHeading: React.SFC<TableHeadingProps> = ({ children }) => { |
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.
FC again
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.
done
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.
its not done yet
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.
done
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 still see const TableHeading: React.SFC
import React from 'react'; | ||
import { TableRowProps } from './types'; | ||
|
||
const TableRow: React.SFC<TableRowProps> = ({ children }) => { |
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.
FC again
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.
here as well
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.
??
import React from 'react'; | ||
import { TbodyProps } from './types'; | ||
|
||
const Tbody: React.SFC<TbodyProps> = ({ children }) => { |
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.
FC again
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.
done
import React from 'react'; | ||
import { TheadProps } from './types'; | ||
|
||
const Thead: React.SFC<TheadProps> = ({ children }) => { |
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.
here too
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.
done
import React from 'react'; | ||
import { TfootProps } from './types'; | ||
|
||
const Tfoot: React.SFC<TfootProps> = ({ children }) => { |
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.
here as well
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.
done
src/components/index.ts
Outdated
|
||
//Table | ||
export { default as Table } from './Elements/Table/Table'; | ||
export { default as Thead } from './Elements/Table/Thead'; |
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 would name it as THead
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.
done
src/components/index.ts
Outdated
export { default as Table } from './Elements/Table/Table'; | ||
export { default as Thead } from './Elements/Table/Thead'; | ||
export { default as Tfoot } from './Elements/Table/Tfoot'; | ||
export { default as Tbody } from './Elements/Table/Tbody'; |
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 would name them as TFoot
and TBody
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.
done
The build is failing because you did not update filenames @fayez-baig |
🎉 This PR is included in version 1.0.19 🎉 The release is available on: Your semantic-release bot 📦🚀 |
No description provided.