Add docs for Table props "expandIconAsCell" and "expandIconColumnIndex"

[kafkahw]

• I have searched the issues of this repository and believe that this is not a duplicate.

What problem does this feature solve?

Props "expandIconAsCell" and "expandIconColumnIndex" are recommended to use for Table component in this issue (#3055). However, there's no docs about those two props.

What does the proposed API look like?

"expandIconAsCell" and "expandIconColumnIndex" show up in https://ant.design/components/table/

[afc163]

expandIconAsCell and expandIconColumnIndex is not good API naming for Table IMO, so it is not documented yet. Maybe we should add another property like showExpandIcon or hideExpandIcon for #3055

[kafkahw]

I like the idea of adding another property called hideExpandIcon. It should be false by default. If user wants to hide the expand icon, they can easily put it onto Table.

<Table
   hideExpandIcon
   {...otherProps}
/>

[IssueHuntBot]

@BoostIO funded this issue with $60. Visit this issue on Issuehunt

[hengkx]

I think showExpandIcon is better. It value can be boolean and number. default true.
When the value is number type, less than 0 does not show icon, greater than or equal to 0 in the specified position show icon.

[hengkx]

#11802
Can we deal with this issue together?

[IssueHuntBot]

@twisger has started working. Visit this issue on Issuehunt

[twisger]

@afc163
I think the api should looks like
expandIcons: { expand: <Icon type="plus" />, collapse: <Icon type="minus" /> }
when it is assigned to null, the expandIcon will hide.

[IssueHuntBot]

@twisger has stopped working. Visit this issue on Issuehunt

[IssueHuntBot]

@Veveue has started working. Visit this issue on Issuehunt

[IssueHuntBot]

@Veveue has started working. See it on IssueHunt

[IssueHuntBot]

@afc163 has stopped working. See it on IssueHunt

[IssueHuntBot]

@rakshitkumarcse has submitted a pull request. See it on IssueHunt

[mighty-phoenix]

expandIconAsCell and expandIconColumnIndex is not good API naming for Table IMO, so it is not documented yet. Maybe we should add another property like showExpandIcon or hideExpandIcon for #3055

I believe the names are just fine. They portray what they suppose. I don't think you should change them.
What do you think @afc163 ?

[IssueHuntBot]

@BoostIO has cancelled funding for this issue.(Cancelled amount: $60.00) See it on IssueHunt