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

Fix/animation to search tables loading issue #1531 #1540

Closed
wants to merge 3 commits into from
Closed

Fix/animation to search tables loading issue #1531 #1540

wants to merge 3 commits into from

Conversation

TheVixhal
Copy link

@TheVixhal TheVixhal commented Feb 10, 2024

Fix/animation to search tables loading issue #1531

/claim #1531

Implemented -

update TableGrid.tsx : timeout={(sectionIndex + 1) * 0 + tableIndex * 0}
update useBasicSearch.tsx : set debounce number 200

Video :

Tables.Test-94d3f.Rowy.dev.-.Google.Chrome.2024-02-11.14-12-23.mp4

Fix/animation to search tables loading issue #1531

/claim #1531

Implemented -

update TableGrid.tsx
commented out this line of code :
 //timeout={(sectionIndex + 1) * 100 + tableIndex * 50}

Video : https://github.com/TheVixhal/Video/blob/751b7962487c866eb4a8dcb7db976793898d52f1/Tables%20%E2%80%A2%20ghoda-94d3f%20%E2%80%A2%20Rowy%20(dev)%20-%20Google%20Chrome%202024-02-11%2003-46-24.mp4
Copy link

vercel bot commented Feb 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rowy-os ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 18, 2024 4:09pm
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
rowy-typedoc ⬜️ Ignored (Inspect) Visit Preview Feb 18, 2024 4:09pm

Copy link

vercel bot commented Feb 10, 2024

@TheVixhal is attempting to deploy a commit to the Rowy Team on Vercel.

A member of the Team first needs to authorize it.

Update useBasicSearch.ts
set debounce number 200
Update TableGrid.tsx
@TheVixhal
Copy link
Author

Hi Sir, ( @bofeiw @harinij )
Kindly review my recent changes. Thanks!

@TheVixhal TheVixhal closed this by deleting the head repository Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant