Skip to content

Commit

Permalink
fix(aggrid): remove box shadow from editor cells (#600)
Browse files Browse the repository at this point in the history
  • Loading branch information
nuke-ellington authored Jun 27, 2023
1 parent 69e69c0 commit e3821b3
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 1 deletion.
4 changes: 4 additions & 0 deletions packages/aggrid/scss/ix-aggrid.scss
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,10 @@
}
}

.ag-cell-inline-editing {
box-shadow: none;
}

.ag-header-cell-resize::after {
transition: background-color 250ms linear;
}
Expand Down
9 changes: 9 additions & 0 deletions packages/aggrid/src/components/tests/aggrid.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,15 @@ regressionTest.describe('aggrid', () => {
expect(await page.screenshot({ fullPage: true })).toMatchSnapshot();
});

regressionTest('editor', async ({ page }) => {
await page.goto('basic.html');

const editorCellHandle = await page.waitForSelector('.ag-cell-not-inline-editing[col-id="price"]');
await editorCellHandle.dblclick();

expect(await page.screenshot({ fullPage: true })).toMatchSnapshot();
});

regressionTest('pagination', async ({ page }) => {
await page.goto('pagination.html');
expect(await page.screenshot({ fullPage: true })).toMatchSnapshot();
Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 1 addition & 1 deletion packages/aggrid/src/components/tests/basic.html
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
const columnDefs = [
{ field: 'make', resizable: true, checkboxSelection: true },
{ field: 'model', resizable: true, sortable: true, filter: true },
{ field: 'price', resizable: true },
{ field: 'price', resizable: true, editable: true },
];

const rowData = [
Expand Down

0 comments on commit e3821b3

Please sign in to comment.