Skip to content

Conversation

@ArtemHoruzhenko
Copy link
Collaborator

What

fixed issue with 2 scrolls appeared for statistics and main container at the same time

Testing

Before
https://github.com/user-attachments/assets/d5780cf8-1190-4666-90d4-5e20d06c176b

After
https://github.com/user-attachments/assets/3923c01c-419a-473d-8d9c-ca6416ca3d9f

@github-actions
Copy link
Contributor

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 82.61% 20573/24904
🟡 Branches 67.76% 8780/12958
🟡 Functions 77.29% 5538/7165
🟢 Lines 83.02% 20143/24264

Test suite run success

5270 tests passing in 689 suites.

Report generated by 🧪jest coverage report action from adde36c

import { FlexItem } from 'uiSrc/components/base/layout/flex'

export const RdiInstancePageContentContainer = styled(FlexItem)`
height: calc(100% - 210px);
Copy link
Member

Choose a reason for hiding this comment

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

Tip: You can leave a note in a comment on what exactly 210 is (I guess it's the header + padding + something else), so we know how to calculate it in case we need to update it in the future.

@ArtemHoruzhenko ArtemHoruzhenko merged commit bbdca7d into main Nov 11, 2025
18 checks passed
@ArtemHoruzhenko ArtemHoruzhenko deleted the fe/feature/RI-7687-adjust-rdi-fix-statistics-double-scroll branch November 11, 2025 14:12
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.

4 participants