Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sample metadata table in the html report is not paginating #191

Closed
davidecarlson opened this issue Nov 8, 2023 · 2 comments · Fixed by #195 or #274
Closed

Sample metadata table in the html report is not paginating #191

davidecarlson opened this issue Nov 8, 2023 · 2 comments · Fixed by #195 or #274
Labels
bug Something isn't working

Comments

@davidecarlson
Copy link
Contributor

Description of the bug

The sample metadata table only lists 10 rows at a time and is supposed to spill over into additional clickable pages if there are more than 10 samples. Currently, this not occurring. The link to click on the next set of samples does not appear below the table.

@pinin4fjords determined that this can be fixed by change dom = 't' to dom = 'tp' in the report template here. I've confirmed that this solves the issue.

I will submit a PR with this change soon and am opening this issue so that I can reference it in the changelog.

Best,
Dave

Command used and terminal output

No response

Relevant files

No response

System information

No response

@davidecarlson davidecarlson added the bug Something isn't working label Nov 8, 2023
@pinin4fjords pinin4fjords linked a pull request Nov 8, 2023 that will close this issue
10 tasks
@davidecarlson davidecarlson reopened this May 23, 2024
@davidecarlson
Copy link
Contributor Author

I just wanted to reopen this issue. It was resolved previously, but it seems like the fix didn't actually make it into the 1.5.0 release, even though it's noted in the changelog.

The file in question seems to be fixed on the dev branch, so maybe there is nothing else that needs to be done, but it's not fixed in the Master branch despite what the release notes say, so I wanted to bring it up anyway.

@pinin4fjords
Copy link
Member

Actually dev and master are the same, it's just that there was a typo in the fix ('tb' rather than 'tp').

I'll apply another fix. Note that you can supply your own amended template via --report_file before such time as this make it into a release.

@pinin4fjords pinin4fjords linked a pull request May 24, 2024 that will close this issue
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants