Skip to content

testing the new algolia search#92

Open
AndreiSirbu wants to merge 13 commits intodevfrom
feature/as_algolia_search_update_1212785600924273
Open

testing the new algolia search#92
AndreiSirbu wants to merge 13 commits intodevfrom
feature/as_algolia_search_update_1212785600924273

Conversation

@AndreiSirbu
Copy link
Collaborator

In this commit I've updated the algolia search modal to include a product dropdown and I need to test it out on dev first.
Screenshot 2026-01-30 at 18 58 32

@github-actions

This comment was marked as resolved.

@AndreiSirbu AndreiSirbu marked this pull request as draft February 2, 2026 16:23
@hilram7 hilram7 assigned hilram7 and unassigned hilram7 Feb 2, 2026
@AndreiSirbu AndreiSirbu force-pushed the feature/as_algolia_search_update_1212785600924273 branch from ebf51ba to e7d892d Compare February 13, 2026 14:47
Hilary Ramirez added 3 commits March 2, 2026 12:36
- Replace single-select product dropdown with multi-select checkboxes in search modal
- Add dedicated search results page at /search with product filtering
- Pressing Enter in search modal now navigates to full results page instead of first result
- Product filter selections persist in localStorage and URL parameters
- Search results page updates dynamically when URL parameters change
- Group results by product with result counts
- Floating button appears when scrolled down more than 300px
- Smooth scroll animation to return to top
- Positioned in bottom-right corner with hover effects
Replace fragile regex with safe DOM-based HTML stripping to prevent
potential HTML injection from incomplete tag fragments.

Resolves CodeQL security warning.
- Add sequential numbering to search results across all products
- Update product headings to show result range (e.g., 'showing 1-15 of 103 results')
- Range updates dynamically as user scrolls and loads more results
- Improves navigation through large result sets
- Use iterative tag removal to handle nested/malformed tags
- Remove remaining angle brackets to prevent partial tag injection
- Addresses CodeQL security warning: js/incomplete-multi-character-sanitization
- Replace IntersectionObserver infinite scroll with Previous/Next pagination buttons
- Add page jump input field allowing users to type and jump to any page
- Add results-per-page selector with options: 25, 50, 100, 150, 200 (default: 25)
- Add jump to bottom button that appears when viewing top of results
- Update result counters to show "X results on this page" instead of global totals
- Add automatic scroll-to-top when changing pages
- Add URL persistence for page number and resultsPerPage settings
- Fix browser back button to restore previous search state and position
- Fix pagination navigation issues when resultsPerPage exceeds default
The code is safe because it uses two mitigation strategies:
1. Iterative loop that repeats replacement until no changes occur
2. Final removal of all angle brackets to prevent any tag fragments

This addresses the false positive CodeQL warning.
Moving the suppression directive to the same line as the flagged code,
as this format is sometimes required by static analysis tools.
…ter sanitization

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@hilram7 hilram7 marked this pull request as ready for review March 3, 2026 16:45
Hilary Ramirez added 2 commits March 3, 2026 12:02
- Add overflow: hidden to both back-to-top and jump-to-bottom buttons
- Adjust down arrow translateY from -6px to -10px
- Ensures both arrow lines touch circle edge with no gaps or overflow
@hilram7 hilram7 deployed to development March 3, 2026 17:09 — with GitHub Actions Active
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.

3 participants