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

Implement #206: Add "Show More" for Table with Changes in Benchmark Set in Compare View #208

Merged
merged 2 commits into from
Jul 16, 2024

Conversation

F12-Syntex
Copy link

Improved the usability of the compare view by introducing a "Show More" link for the "Changes in Benchmark Set" table. Key enhancements include:

  • Initial Display: The table now initially displays only 3 entries to prevent overwhelming the user with large datasets.
  • Show More Link: Added a "Show More" link below the table, allowing users to expand and view all entries on demand.
  • CSS Class: Introduced a .hidden class in style.css to manage the visibility of additional table rows.
  • JavaScript Function: Implemented a showAllResults function to handle the display of all entries when the "Show More" link is clicked.

@F12-Syntex
Copy link
Author

@smarr bug fixed, and improved the show more feature into a collapsable section.

@smarr smarr force-pushed the master branch 6 times, most recently from 19826b3 to c055b99 Compare July 11, 2024 16:16
F12-Syntex and others added 2 commits July 16, 2024 09:29
This simplifies the implementation by using the nth-child pseudo class. This way, we do not need to loop over rows.

It also removes unused CSS classes and changes the code style to fit more with the existing code.
 - CSS classes use “kebab-case”.
 - use JQuery to match existing code
 - remove the .showMore which was already unsued

Signed-off-by: Stefan Marr <[email protected]>
@smarr smarr changed the title Fix #206: Add Show More Functionality and Default Filtering for Missing Data in Compare View Implement #206: Add "Show More" for Table with Changes in Benchmark Set in Compare View Jul 16, 2024
@smarr smarr added the enhancement New feature or request label Jul 16, 2024
@smarr smarr merged commit 2d2e1c1 into smarr:master Jul 16, 2024
1 check passed
@smarr smarr linked an issue Jul 16, 2024 that may be closed by this pull request
2 tasks
@smarr
Copy link
Owner

smarr commented Jul 16, 2024

@F12-Syntex thanks for implementing this! Almost forgot to merge it. Sorry :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compare View, Usability: Filter Missing Data per CSS per default
2 participants