Skip to content

Conversation

@CatalinB7
Copy link
Collaborator

@CatalinB7 CatalinB7 commented Sep 26, 2022

adds support for multiple expanded rows

@caleniuc caleniuc force-pushed the feat/expandable_rows branch from 9673743 to 7f790b2 Compare September 29, 2022 07:46
@caleniuc caleniuc force-pushed the feat/expandable_rows branch 2 times, most recently from fb23e13 to 9810f70 Compare October 3, 2022 13:40
@caleniuc caleniuc changed the title feat(grid): wip feat(grid): multiple expanded rows Oct 3, 2022
@caleniuc caleniuc force-pushed the feat/expandable_rows branch from 9810f70 to b46835f Compare October 3, 2022 14:36
const firstRow = fixture.debugElement.query(By.css('.ui-grid-row'));
firstRow.nativeElement.dispatchEvent(EventGenerator.click);
expect(grid.rowClick.emit).toHaveBeenCalledWith(jasmine.objectContaining({
row: data[0],
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

@caleniuc caleniuc force-pushed the feat/expandable_rows branch from b46835f to 3a6bd9f Compare October 3, 2022 15:25
@caleniuc caleniuc force-pushed the feat/expandable_rows branch from 3a6bd9f to 14b7bb3 Compare October 3, 2022 15:46
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 14b7bb3 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 85.7% (50% is the threshold).

This pull request will bring the total coverage in the repository to 92.7% (0.0% change).

View more on Code Climate.

@caleniuc caleniuc merged commit a578537 into master Oct 4, 2022
@caleniuc caleniuc deleted the feat/expandable_rows branch October 4, 2022 05:00
gheorghitahurmuz pushed a commit that referenced this pull request Dec 12, 2023
feat(grid): multiple expanded rows
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants