-
Notifications
You must be signed in to change notification settings - Fork 88
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
style(line-profile): table formatting for line metadata #568
Conversation
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.
Thanks! It looks amazing 👍
src/pages/ProfileLineDetails.tsx
Outdated
} | ||
|
||
& .line-details { | ||
height: 2rem; |
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.
That could be a problem when the lines are lengthy (especially on mobile devices).using line-height and padding would be better
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.
Updated the code and also checked on mobile (there the text is too big and the line will collapse and just take more space).
So now this is also fully mobile responsive.
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.
This is continue of #543
I changed the section where we display data on the line to look something like the table bellow regarding colors just to be consist.