-
Notifications
You must be signed in to change notification settings - Fork 363
Display MethodDetails content on smaller screens #3309
Conversation
CLA Assistant Lite All Contributors have signed the CLA. |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
Pull Request Test Coverage Report for Build 1712268936
💛 - Coveralls |
Deployment links
|
E2E Tests Failed Failed tests:
|
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.
Looking good @usame-algan, but should it not also wrap Value
on line 56? And does the scroll issue only affect contract interactions?
For some value types we don't set flex box so the value gets displayed in their own row but it wouldn't hurt to still set the min-width to 50% even if there is only one column. The scroll should "trigger" every time there is not enough available space within |
Nvm, I saw it on the link you provided. IMHO, the value and label should scroll separately if possible. Also there's plenty of vertical space, yet the label tries to be on a single line. Might be worth adding |
I agree it would be better to use the available vertical space in those cases. I added |
@@ -13,6 +13,11 @@ const NestedWrapper = styled.div` | |||
padding-left: 4px; | |||
` | |||
|
|||
export const ValueWrapper = styled.div` |
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.
I'd move this to MethodDetails
as it is being used there and not here.
In combination with |
What it solves
Resolves #3284
How this PR fixes it
Adds line breaks to labels and values within
MethodDetails
if there is not enough space left and adds horizontal scroll.How to test it
MethodDetails
data is visible/scrollable on smaller screensScreenshots