-
-
Notifications
You must be signed in to change notification settings - Fork 74
Conversation
@@ -128,7 +128,7 @@ export default class CellFactory { | |||
} | |||
} | |||
|
|||
private handleChange = (idx: number, i: number, e: any) => { | |||
private handleChange = (idx: number, i: number, value: any) => { |
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.
Doing away with the event wrapper, it served no purpose at this point
DashTable.getCell(0, 8).within(() => { | ||
cy.get('.Select-value-label').should('have.html', expectedValue); | ||
}); | ||
}); |
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.
Click on a cell with dropdown, check current value, select a different value, make sure it's applied.
@@ -45,11 +45,13 @@ export default class ControlledTable extends Component<ControlledTableProps> { | |||
|
|||
// Fallback method for paste handling in Chrome | |||
// when no input element has focused inside the table | |||
window.addEventListener('resize', this.handleResize); |
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.
Zoom triggers a resize event, so does resizing the window/browser.. but that's the best we have access 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.
fix dropdown behavior on click (saves selection)
additional test for dropdown
fix border style on active cell
resize on zoom change