-
Notifications
You must be signed in to change notification settings - Fork 0
feat: animations, transitions, and UI enhancements #24
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
Merged
Merged
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
0f1bec2
fix: smooth gradient transition on navbar CTA using overlay pseudo-el…
Smartlify07 9403570
fix: smooth gradient transition on hero book consultation button
Smartlify07 a057b10
feat: scroll-into-view expand animation on hero image using motion
Smartlify07 87b105c
fix: hero image expands from 700px to full 1024px container on scroll
Smartlify07 9aac4c7
refactor: use scale transform for hero image scroll animation instead…
Smartlify07 61192d6
chore: add motion package and finish navbar animation
Smartlify07 bda2bae
feat: horizontal expand/shrink on bridging the gap cards on hover
Smartlify07 8b474c4
fix: add shrink-0 to image wrappers to prevent flex compression
Smartlify07 a4cc51f
fix: testimonials image overflow and scroll-to-end on button click
Smartlify07 66137c0
fix: horizontal overflow clipping on testimonials scroll container
Smartlify07 9e97af0
fix: disable manual horizontal scroll, button-only navigation
Smartlify07 6f6448b
feat: accordion FAQ with expand/collapse and 360deg plus spin
Smartlify07 98ecafe
fix: slow cross rotation speed by 2x (600ms)
Smartlify07 116b00d
feat: add arrow rotation on hover to CTA button
Smartlify07 441c87c
feat: join button scale up, gradient, and arrow rotation on hover
Smartlify07 4c60662
feat: add muted hover background to course cards
Smartlify07 a1a0d49
feat: add gradient overlay and arrow rotation to enroll button
Smartlify07 f5a804c
feat: add gradient border to enroll button using before pseudo-element
Smartlify07 d0709b0
fix: revert hero enroll button; add gradient border to content sectio…
Smartlify07 8758cb7
fix: gradient border only visible on hover, white bg + primary text +…
Smartlify07 39fbc85
fix: add variant outline and isolate to fix text visibility and z-ind…
Smartlify07 3f4290f
fix: make Button background transparent so before gradient border is …
Smartlify07 2cc3e1c
fix: use wrapper pattern for gradient border on enroll button
Smartlify07 91f17c9
feat: add accordion transitions to course modules, gradient hover to …
Smartlify07 dedee60
feat: restrict mission hover interaction to large screens only
Smartlify07 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
🩺 Stability & Availability | 🟠 Major | ⚡ Quick win
Custom accordion drops native disclosure semantics — add
aria-expanded/aria-controls.The previous
<details>/<summary>implementation was natively accessible (announces expanded/collapsed state). The new<button>-driven toggle has noaria-expandedon the button and noaria-controls/idlinking it to the content panel, so screen reader users get no indication of state.♿ Proposed fix
📝 Committable suggestion
🤖 Prompt for AI Agents