Skip to content

Commit

Permalink
Fixed bug which wouldn't let user preview more than 5 lines of an ana…
Browse files Browse the repository at this point in the history
…lysis tabular result file by removing max-height of the table
  • Loading branch information
deepsidhu85 committed Feb 3, 2021
1 parent 2005cf3 commit c0d816f
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 3 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
Changes
=======

21.01 to 20.05
--------------
*[UI]: Fixed bug which was preventing a user from viewing more than 5 rows of a tabular result file. (21.01.1)

20.09 to 21.01
--------------
* [UI]: Fixed bug where sequencing runs could not be deleted on sequencing runs details page. (20.09.1)
Expand Down
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
<groupId>ca.corefacility.bioinformatics</groupId>
<artifactId>irida</artifactId>
<packaging>war</packaging>
<version>21.01</version>
<version>21.01.1</version>
<name>irida</name>
<url>http://www.irida.ca</url>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ import styled from "styled-components";
import { OutputFileHeader } from "../../../components/OutputFiles/OutputFileHeader";

const TabularOutputWrapper = styled.div`
max-height: 300px;
width: 100%;
margin-bottom: ${SPACE_XS};
`;
Expand Down Expand Up @@ -58,7 +57,10 @@ export function AnalysisTabularPreview({ output }) {
pagination={
fileRows.length <= MAX_TABLE_ROWS_PER_PAGE
? false
: { pageSize: MAX_TABLE_ROWS_PER_PAGE }
: {
defaultPageSize: MAX_TABLE_ROWS_PER_PAGE,
pageSizeOptions: [5, 10, 20, 50, 100],
}
}
/>
</TabularOutputWrapper>
Expand Down

0 comments on commit c0d816f

Please sign in to comment.