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

Filter performance and UI touchups #81

Merged
merged 14 commits into from
Apr 4, 2024
Merged

Filter performance and UI touchups #81

merged 14 commits into from
Apr 4, 2024

Conversation

thebeyondr
Copy link
Member

use pre-processing, maps, sets and restructuring to improve filtering performance at scale, and add a few cosmetic changes

@thebeyondr thebeyondr self-assigned this Apr 3, 2024
Copy link

netlify bot commented Apr 3, 2024

Deploy Preview for vd-dev ready!

Name Link
🔨 Latest commit 7beeb96
🔍 Latest deploy log https://app.netlify.com/sites/vd-dev/deploys/660da9cb3763c10008869777
😎 Deploy Preview https://deploy-preview-81--vd-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@CJ-Rose
Copy link
Member

CJ-Rose commented Apr 3, 2024

i think these look not great

Screenshot 2024-04-03 at 11 54 12 AM Screenshot 2024-04-03 at 11 54 42 AM

proposing to change to these

Screenshot 2024-04-03 at 11 55 41 AM Screenshot 2024-04-03 at 11 55 19 AM

thoughts? i can just commit the update here.

@thebeyondr
Copy link
Member Author

thoughts? i can just commit the update here.

yes, looks good to me, thanks!

@thebeyondr thebeyondr merged commit 4583635 into dev Apr 4, 2024
7 checks passed
@thebeyondr thebeyondr deleted the hotfix/perf branch April 4, 2024 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants