-
Notifications
You must be signed in to change notification settings - Fork 0
[MERGE] Refacto/design #57
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request implements a major design refactoring that elevates the visual experience to a "Level 10" premium aesthetic. The changes introduce modern web technologies including Three.js particle backgrounds, smooth scrolling with Lenis, and advanced interactive effects.
Key Changes:
- Introduced Three.js-powered interactive particle network background system
- Implemented Lenis smooth scrolling and GSAP animations
- Refactored visual effects to include 3D tilt, spotlight, and magnetic button interactions
- Created template system for reusable HTML components (loader, head-common, matomo)
- Updated CSS design tokens to a more vibrant, premium color palette
- Cleaned up test files by removing console.spy() patterns for cleaner test output
- Restructured HTML pages with critical inline loaders for zero-flash loading
Reviewed changes
Copilot reviewed 37 out of 38 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updated build script message |
| frontend/tests/unit_test/*.test.js | Removed console spy assertions for cleaner test output |
| frontend/tests/playwright/contact.spec.js | Added cookie and connection errors to non-critical filter list |
| frontend/templates/*.html | New template system for loader, head elements, and analytics |
| frontend/*.html | Major restructuring with inline critical loaders and Three.js integration |
| frontend/components/footer.html | Updated professional description and removed attribution line |
| frontend/components/background.html | Removed (replaced by Three.js system) |
| frontend/assets/js/visual-effects.js | Complete rewrite with tilt, spotlight, and magnetic effects |
| frontend/assets/js/three-bg.js | New Three.js particle network background |
| frontend/assets/js/smooth-scroll.js | New Lenis smooth scrolling implementation |
| frontend/assets/js/modules/*.js | Removed console.log statements and improved navigation handling |
| frontend/assets/css/modules/*.css | Updated design tokens, added premium components, new animations |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 37 out of 38 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
No description provided.