-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Security Solution][Timeline] Fix User not able to scroll down and access Alert table on adding long content in Timeline's Description #101486
[Security Solution][Timeline] Fix User not able to scroll down and access Alert table on adding long content in Timeline's Description #101486
Conversation
@monina-n, could you take a look and let me know if my solution makes sense and how many lines we shall truncate the description on the "Timelines list"? |
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
@elasticmachine merge upstream |
1 similar comment
@elasticmachine merge upstream |
x-pack/plugins/security_solution/public/common/components/line_clamp/index.tsx
Outdated
Show resolved
Hide resolved
2b59d32
to
813e073
Compare
813e073
to
c0599fb
Compare
💚 Build SucceededMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
…cess Alert table on adding long content in Timeline's Description (elastic#101486) * Add LineClamp component to timeline description * Truncate timeline description on timeline table * Fix StyledLineClamp styled component performance issue Read more: https://styled-components.com/docs/faqs#why-should-i-avoid-declaring-styled-components-in-the-render-method
…cess Alert table on adding long content in Timeline's Description (#101486) (#102301) * Add LineClamp component to timeline description * Truncate timeline description on timeline table * Fix StyledLineClamp styled component performance issue Read more: https://styled-components.com/docs/faqs#why-should-i-avoid-declaring-styled-components-in-the-render-method
Summary
issue: #100259
1. Open timeline page
When the timeline description is too big, it takes too much space leaving almost no room for the list of events.
To solve this problem I reused the component that is used to collapse the Event description.
After change:
Initial state (collapsed)
Expanded
2. Timelines page
A long description also affects the "Timelines page", where the description cell takes too much space.
To solve this second problem, we are truncating the content in 5 lines.
After change
Checklist