-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
fix: Fix pod log viewer scrollbars #14199
Conversation
Signed-off-by: Alex Collins <alex_collins@intuit.com>
Signed-off-by: Alex Collins <alex_collins@intuit.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
YOLO Approve.
@crenshaw-dev can I bump review pls? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Local test confirms scrolls are scrollier.
@alexec can you check the CI lint issue? |
/cherry-pick release-2.8 |
/cherry-pick release-2.7 |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #14199 +/- ##
==========================================
- Coverage 49.73% 49.72% -0.01%
==========================================
Files 261 261
Lines 44705 44705
==========================================
- Hits 22234 22231 -3
- Misses 20284 20286 +2
- Partials 2187 2188 +1 ☔ View full report in Codecov by Sentry. |
* fix: Fix pod log viewer scrollbars Signed-off-by: Alex Collins <alex_collins@intuit.com> * fix scrolling Signed-off-by: Alex Collins <alex_collins@intuit.com> --------- Signed-off-by: Alex Collins <alex_collins@intuit.com>
* fix: Fix pod log viewer scrollbars Signed-off-by: Alex Collins <alex_collins@intuit.com> * fix scrolling Signed-off-by: Alex Collins <alex_collins@intuit.com> --------- Signed-off-by: Alex Collins <alex_collins@intuit.com>
* fix: Fix pod log viewer scrollbars * fix scrolling --------- Signed-off-by: Alex Collins <alex_collins@intuit.com> Co-authored-by: Alex Collins <alexec@users.noreply.github.com> Signed-off-by: schakrad <58915923+schakrad@users.noreply.github.com>
* fix: Fix pod log viewer scrollbars Signed-off-by: Alex Collins <alex_collins@intuit.com> * fix scrolling Signed-off-by: Alex Collins <alex_collins@intuit.com> --------- Signed-off-by: Alex Collins <alex_collins@intuit.com>
* fix: Fix pod log viewer scrollbars Signed-off-by: Alex Collins <alex_collins@intuit.com> * fix scrolling Signed-off-by: Alex Collins <alex_collins@intuit.com> --------- Signed-off-by: Alex Collins <alex_collins@intuit.com>
List
. You must useGrid
.scrollToBottom
when the user tries to scroll to the bottom.