Skip to content
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

feat(web): Novu Dashboard fix- After clearing the search a provider input selected tab list should be visible not to show list from begining #6386

Merged

Conversation

Arunshaik2001
Copy link
Contributor

@Arunshaik2001 Arunshaik2001 commented Aug 24, 2024

What changed? Why was the change needed?

fixes #6385

Clearing the provider input text make the list render from beginning of the list irrespective of selected tab.

calling scrollToElement on selected tab after getting empty string.

Screenshots

novu-sidebar-fix.mp4
Expand for optional sections

Special notes for your reviewer

I've added timeout of 1000ms so that providers list is first rendered and then scrolled to selected tab.

@Arunshaik2001 Arunshaik2001 requested a review from a team August 24, 2024 10:12
@Arunshaik2001 Arunshaik2001 changed the title (Novu Dashboard fix): After clearing the search a provider input selected tab list should be visible not to show list from begining feat(scope): Novu Dashboard fix- After clearing the search a provider input selected tab list should be visible not to show list from begining Aug 24, 2024
@Arunshaik2001 Arunshaik2001 changed the title feat(scope): Novu Dashboard fix- After clearing the search a provider input selected tab list should be visible not to show list from begining feat(fix): Novu Dashboard fix- After clearing the search a provider input selected tab list should be visible not to show list from begining Aug 24, 2024
@Arunshaik2001 Arunshaik2001 changed the title feat(fix): Novu Dashboard fix- After clearing the search a provider input selected tab list should be visible not to show list from begining feat(web): Novu Dashboard fix- After clearing the search a provider input selected tab list should be visible not to show list from begining Aug 24, 2024
@jainpawan21 jainpawan21 enabled auto-merge (squash) October 7, 2024 11:09
@jainpawan21 jainpawan21 requested a review from BiswaViraj October 7, 2024 11:09
@jainpawan21 jainpawan21 merged commit ed7aa47 into novuhq:next Oct 20, 2024
8 of 11 checks passed
jainpawan21 pushed a commit that referenced this pull request Oct 20, 2024
…nput selected tab list should be visible not to show list from begining (#6386)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants