Jithen Shriyan
Jithen Shriyan
> Currently if you have many insights (so that there are 2 or more pages on the saved insights page) you will not be able to see insights from a...
> 1. The insights need to be paginated or we need to adapt the flow for a case when the user has many insights. Yes, pagination makes sense. Just implemented...
> * The "add text card" dropdown is missing from the button (see inline comment) Oh yes, sorry about that just added it back. I had originally thought it was...
> 3\. Not a total blocker, but would be nice to have solved: If there are many insights the list displays a scrollbar, but there is no gap between the...
@thmsobrmlr just wanted to check in. How is it looking?
Hi, I'd like to contribute to fixing this bug.
@pauldambra I can take a look at this, feel free to assign me.
@pauldambra sounds good!
@pauldambra started a draft PR for this would love to get some feedback. I think the two main things left to address are adding some test cases and also (if...
@pauldambra just opened up #19802 for review! Feel free to take a look. Thanks!