fix(PaginatedStorage): controls should be fixed#2217
Conversation
| padding-right: 40px; | ||
| padding-left: 20px; | ||
|
|
||
| transform: translateX(-20px); |
There was a problem hiding this comment.
Could you please explain how it works and where 40 and 20 px come from?
mb move them to css variables for clearance
There was a problem hiding this comment.
In short, it's black magic :)
If serious, we have our scroll container ydb-cluster with side paddings. And thats the trouble. So, I just mislead this paddings (we have similar styles for tabs). Maybe we should rework layout for all cluster page, what do u think?
There was a problem hiding this comment.
I think if current solution fixes current problem - we could merge it but as well we could create task to techdebt with description of what should be reworked
I dont think for now we have that much of a resource for massive reworks of that type
However sometimes it could be done
I checked this issue with many pdisks and vdidks with mocks from this pr #1793 and it seems to work
2 participants
closes #2195
Stand
CI Results
Test Status:⚠️ FLAKY
📊 Full Report
😟 No changes in tests. 😕
Bundle Size: ✅
Current: 83.37 MB | Main: 83.37 MB
Diff: +1.35 KB (0.00%)
✅ Bundle size unchanged.
ℹ️ CI Information