Skip to content

Commit

Permalink
Disable the <RoundSize> component wrapper (#1333)
Browse files Browse the repository at this point in the history
Previously, a bug in chrome broke scrolling in the table.

As demonstrated in this fiddle, the bug has been fixed.
https://jsfiddle.net/2rmz7p1d/5/

The RoundSize workaround has non-zero performance impact,
so we should disable it now that chrome is fixed.
  • Loading branch information
themadcreator authored Jul 10, 2017
1 parent d5627ed commit e4cde27
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions packages/table/src/headers/rowHeader.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ import * as React from "react";

import * as Classes from "../common/classes";
import { IRowIndices } from "../common/grid";
import { RoundSize } from "../common/roundSize";
import { IClientCoordinates } from "../interactions/draggable";
import { IIndexedResizeCallback } from "../interactions/resizable";
import { Orientation } from "../interactions/resizeHandle";
Expand Down Expand Up @@ -112,11 +111,9 @@ export class RowHeader extends React.Component<IRowHeaderProps, {}> {
}

return (
<RoundSize>
<div style={style}>
{cells}
</div>
</RoundSize>
<div style={style}>
{cells}
</div>
);
}

Expand Down

1 comment on commit e4cde27

@blueprint-bot
Copy link

Choose a reason for hiding this comment

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

Disable the `` component wrapper (#1333)

Preview: documentation
Coverage: core | datetime

Please sign in to comment.