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

feat: rows/page dropbox. Fixes #1764 #2337

Merged
merged 3 commits into from
Jan 23, 2025
Merged

Conversation

SaniyaKalamkar
Copy link
Contributor

@SaniyaKalamkar SaniyaKalamkar commented Jan 19, 2025

Fixes #1764. Created rows per page option so events being displayed is not fixed at 6.
Edit: Changed style of dropbox:
Screenshot 2025-01-21 at 10 13 02 PM

@SaniyaKalamkar SaniyaKalamkar marked this pull request as draft January 19, 2025 01:32
@SaniyaKalamkar SaniyaKalamkar force-pushed the new-navpages branch 2 times, most recently from 7cf6782 to 4750e3a Compare January 19, 2025 01:36
@vigith vigith marked this pull request as ready for review January 19, 2025 02:46
Copy link

codecov bot commented Jan 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 57.85%. Comparing base (c5f8852) to head (eac9e63).
Report is 204 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2337      +/-   ##
==========================================
- Coverage   58.21%   57.85%   -0.37%     
==========================================
  Files         402      469      +67     
  Lines       28061    55296   +27235     
==========================================
+ Hits        16337    31991   +15654     
- Misses      10790    22227   +11437     
- Partials      934     1078     +144     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vigith vigith enabled auto-merge (squash) January 19, 2025 17:05
@vigith vigith disabled auto-merge January 21, 2025 04:48
@SaniyaKalamkar SaniyaKalamkar force-pushed the new-navpages branch 2 times, most recently from 0680aa3 to 2c7c527 Compare January 22, 2025 03:20
@veds-g
Copy link
Contributor

veds-g commented Jan 22, 2025

for the <TableContainer> can we have the maxHeight: "75%" instead of fixed 60rem, so it has greater height on bigger displays.

Copy link
Contributor

@veds-g veds-g left a comment

Choose a reason for hiding this comment

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

LGTM! Awesome work 👍🏻

@vigith vigith merged commit 8cd95a8 into numaproj:main Jan 23, 2025
25 checks passed
@vigith
Copy link
Member

vigith commented Jan 23, 2025

Thank you for the contribution 🎉

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.

UI: K8s Pipeline Events improved page navigation
5 participants