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

Table: add row-style prop. closed #1251 #1348

Merged
merged 1 commit into from
Nov 25, 2016

Conversation

furybean
Copy link
Contributor

@furybean furybean commented Nov 24, 2016

Please makes sure these boxes are checked before submitting your PR, thank you!

  • Make sure you follow Element's Contributing Guide.
  • Make sure you are merging your commits to dev branch.
  • Rebase before creating a PR to keep commit history clear.
  • Add some descriptions and refer relative issues for you PR.

#1251

@@ -1183,7 +1183,8 @@ Customize table column so it can be integrated with other components.
| fit | whether width of column automatically fits its container | boolean | — | true |
| show-header | whether table header is visible | boolean | - | true |
| highlight-current-row | whether current row is highlighted | boolean | — | false |
| row-class-name | function that returns custom class names for a row | Function(row, index) | — | — |
| row-class-name | function that returns custom class names for a row. If its value is a string, the value will be every row's class name | Function(row, index)/String | — | — |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If its value is a string, the value will be every row's class name
=>
If the returned value is a string, it will apply to every row

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

我说的好像并不是这个意思,我的意思是:可以是 Function,也可以是 string;如果是 string,则会对每一行加一个固定的 class。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

哦哦,那就 function that returns custom class names for a row, or a string assigning class names for every row,下面也类似吧

@@ -1183,7 +1183,8 @@ Customize table column so it can be integrated with other components.
| fit | whether width of column automatically fits its container | boolean | — | true |
| show-header | whether table header is visible | boolean | - | true |
| highlight-current-row | whether current row is highlighted | boolean | — | false |
| row-class-name | function that returns custom class names for a row | Function(row, index) | — | — |
| row-class-name | function that returns custom class names for a row. If its value is a string, the value will be every row's class name | Function(row, index)/String | — | — |
| row-style | function that returns custom style for a row. If its value is a object, the value will be every row's style | Function(row, index)/Object | — | — |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If its value is a object, the value will be every row's style
=>
If the returned value is an object, it will apply to every row

@Leopoldthecoder Leopoldthecoder merged commit 812a953 into ElemeFE:dev Nov 25, 2016
@furybean furybean deleted the row-style branch November 30, 2016 04:38
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 this pull request may close these issues.

3 participants