Foundry376/Mailspring
1
1
Fork 0
mirror of https://github.com/Foundry376/Mailspring.git synced 2025-02-23 23:54:13 +08:00
Code Issues Projects Releases Packages Wiki Activity

fix(table): Fix row rendering (also fixes focus)

Browse source 
- When rendering lazy list, need to pass same reference to the renderer
or otherwise a new component will always be mounted
This commit is contained in:
Juan Tejada 2016-04-29 14:54:54 -07:00
parent 55bd04b3a2
commit 7d09c109ac
1 changed files with 19 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

37
src/components/table.jsx
View file

@ -98,24 +98,25 @@ export default class Table extends Component {
CellRenderer: TableCell,
}
renderBody() {
const {tableData, rowHeight, bodyHeight, displayNumbers, displayHeader, extraProps, RowRenderer, CellRenderer} = this.props
const rows = displayHeader ? tableData.rows.slice(1) : tableData.rows
renderRow = ({idx}) => {
const {tableData, displayNumbers, displayHeader, extraProps, RowRenderer, CellRenderer} = this.props
const rowIdx = displayHeader ? idx + 1 : idx;
return (
<RowRenderer
key={`row-${rowIdx}`}
rowIdx={rowIdx}
displayNumbers={displayNumbers}
tableData={tableData}
extraProps={extraProps}
CellRenderer={CellRenderer}
{...extraProps}
/>
)
}
const itemRenderer = ({idx}) => {
const rowIdx = displayHeader ? idx + 1 : idx;
return (
<RowRenderer
key={`row-${rowIdx}`}
rowIdx={rowIdx}
displayNumbers={displayNumbers}
tableData={tableData}
extraProps={extraProps}
CellRenderer={CellRenderer}
{...extraProps}
/>
)
}
renderBody() {
const {tableData, rowHeight, bodyHeight, displayHeader} = this.props
const rows = displayHeader ? tableData.rows.slice(1) : tableData.rows
return (
<LazyRenderedList
@ -123,7 +124,7 @@ export default class Table extends Component {
itemHeight={rowHeight}
containerHeight={bodyHeight}
BufferTag="tr"
ItemRenderer={itemRenderer}
ItemRenderer={this.renderRow}
RootRenderer="tbody"
/>
)