Skip to content

Conversation

@Naveenkumar27
Copy link
Contributor

Added table-row expansion feature. It expands the row of a table on clicking the "View Details" button to show the details of an instance such as docker Id and labels. The feature is implemented using Angular-Material table expand-row feature.

@ghost ghost assigned Naveenkumar27 Jan 23, 2019
@ghost ghost added the review label Jan 23, 2019
@codecov-io
Copy link

codecov-io commented Jan 24, 2019

Codecov Report

Merging #104 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           develop    #104   +/-   ##
=======================================
  Coverage     4.69%   4.69%           
=======================================
  Files           11      11           
  Lines          213     213           
  Branches        15      15           
=======================================
  Hits            10      10           
  Misses         203     203

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 74128f6...14316fd. Read the comment docs.

Copy link
Contributor

@janniclas janniclas left a comment

Choose a reason for hiding this comment

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

image
As seen in the image there are two visual issues left. On the one hand the button when clicked is half surrounded with this blue line.
And after adding the view details button the table is overflowing to the right, which is not desirable.

Copy link
Contributor

@janniclas janniclas left a comment

Choose a reason for hiding this comment

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

please update the code according to the added comments

@janniclas janniclas merged commit fae6903 into develop Jan 27, 2019
@ghost ghost removed the review label Jan 27, 2019
@Naveenkumar27 Naveenkumar27 deleted the feature/tableExpansion branch January 31, 2019 10:50
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.

4 participants